search

nulib / arch

Northwestern University institutional repository, built on Samvera's Hyrax gem.
2 stars 0 forks source link

update sufia to latest 7.x #48

Closed csyversen closed 7 years ago

csyversen commented 7 years ago

we're running the 7.0 beta, now is a good time to bump up to the latest version

davidschober commented 7 years ago

I think this is the first in line. It will likely have some cascading effects.

On Jan 4, 2017, at 4:41 PM, Chris Syversen notifications@github.com wrote:

we're running the 7.0 beta, now is a good time to bump up to the latest version

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nulib/institutional-repository/issues/48, or mute the thread https://github.com/notifications/unsubscribe-auth/AElKnrxdTXLHAHchphitz2QrWWu_KMimks5rPCARgaJpZM4LbJd6.

csyversen commented 7 years ago

i followed the sufia upgrade instructions and it's mostly ok, but at least two of our view overrides (_controls.html.erb and _footer.html.erb) aren't being picked up only in the root route. On every other page those two overrides are respected and everything looks fine.

I'll probably have to ask in the hydra slack channel what's up

davidschober commented 7 years ago

Note @csyversen will complete this before the big push (~01/13/2017)

davidschober commented 7 years ago

note, is the thumbnail using openseadragon as a viewer for images in that space? If so we might need to rethink this a bit.

csyversen commented 7 years ago

ok, need to push the latest changes to nufiaweb-s!

davidschober commented 7 years ago

Chris, do you want Jon Dorr/ Chris Diaz to go over the existing issues to reverify them?

On Jan 12, 2017, at 4:04 PM, Chris Syversen notifications@github.com wrote:

ok, need to push the latest changes to nufiaweb-s!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nulib/institutional-repository/issues/48#issuecomment-272298620, or mute the thread https://github.com/notifications/unsubscribe-auth/AElKnoS0SC-3tabjay6jHoq6VibbJP6tks5rRqOIgaJpZM4LbJd6.

csyversen commented 7 years ago

@davidschober after this gets deployed, yeah that'd be great

csyversen commented 7 years ago

@davidschober done and deployed! @johndorr @cpd3149 this is ready to be tested!