nulib / avalon

Variations-on-Video Hydra app
Apache License 2.0
3 stars 0 forks source link

two arrows showing up on playlist #353

Closed davidschober closed 6 years ago

davidschober commented 6 years ago

to recreate go to a playlist:

https://canvas.northwestern.edu/courses/47583/modules/items/921326

and note that there are two playlist arrows.

screen shot 2018-02-06 at 11 56 37 am

mbklein commented 6 years ago

Fix pushed to nulib/avalon@1dbec62e7d6c48bf9bf5ed3788f820d1c897af18

davidschober commented 6 years ago

Done. Feel free to push @mbklein

adamjarling commented 6 years ago

@davidschober Verify?

davidschober commented 6 years ago

still seeing it.

On Feb 16, 2018, at 11:08 AM, Adam J. Arling notifications@github.com wrote:

@davidschober https://github.com/davidschober Verify?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nulib/avalon/issues/353#issuecomment-366296916, or mute the thread https://github.com/notifications/unsubscribe-auth/AElKngJ10mTSk6J5DNeQR9OAvhqEfjroks5tVbXwgaJpZM4R7eVI.

davidschober commented 6 years ago

I'm still seeing this

screen shot 2018-02-16 at 11 09 25 am

davidschober commented 6 years ago

Sure. It looks like it's happening on all playlists

https://media.northwestern.edu/playlists/18

to recreate:

  1. sign in
  2. create a playlist
  3. see playlist has two arrows.
adamjarling commented 6 years ago

Ok, I'll run this locally and see what's going on.

adamjarling commented 6 years ago

@davidschober @mbklein From the git commit history, looks like the https://github.com/nulib/avalon/commit/1dbec62e7d6c48bf9bf5ed3788f820d1c897af18 never made it to aws-deploy-prod?

adamjarling commented 6 years ago

There are some other UI items I'm cleaning up while in the code though...

carrickr commented 6 years ago

@davidschober Deployed to prod, not seeing two errors here:

https://media.northwestern.edu/playlists/95

davidschober commented 6 years ago

done