Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade arch to sufia 7.3 #160

Closed
5 tasks done
davidschober opened this issue Mar 24, 2017 · 11 comments
Closed
5 tasks done

upgrade arch to sufia 7.3 #160

davidschober opened this issue Mar 24, 2017 · 11 comments
Assignees

Comments

@davidschober
Copy link
Contributor

davidschober commented Mar 24, 2017

done looks like

  • arch s is upgraded to 7.3
  • current UI customizations are ported if necessary @adamjarling
  • arch-s regressison testing done by @cpd3149
  • Sign off from @cpd3149 to move to production
  • move to production
@davidschober
Copy link
Contributor Author

@nulib/repodev verify the checklist. Also, can we tag any issues we think might get fixed so @cpd3149 knows to verify and close? I started a couple.

@adamjarling
Copy link
Contributor

@davidschober @bmquinn @kdid I'll pull down the 7.3 branch and try rebasing my commits from this week. If anything weird pops up, I'll let you know on Monday.

@bmquinn bmquinn mentioned this issue Mar 27, 2017
@bmquinn bmquinn self-assigned this Mar 27, 2017
This was referenced Mar 27, 2017
@chrisdaaz
Copy link

currently testing. it seems to be working, overall, but there's a weird span element

class="state state-deposited"

displaying between the Title and Abstract: https://nufiaweb-s.library.northwestern.edu/concern/generic_works/sq87bt648

image

this appears on new works since the upgrade:
https://nufiaweb-s.library.northwestern.edu/concern/generic_works/x346d4190
https://nufiaweb-s.library.northwestern.edu/concern/generic_works/xk81jk36q

also, can someone else test the "Batch Create" function? i keep trying to add multiple files through this method, but i keep getting the "!" notification saying i haven't added files.

image

i don't know if this related to the upgrade. everything else looks good to me!

@kdid
Copy link
Contributor

kdid commented Apr 4, 2017

Yes, batch upload not working. Looking into it.

@kdid
Copy link
Contributor

kdid commented Apr 4, 2017

Here is an issue related to "deposited" showing.

samvera-deprecated/sufia#3107

@kdid
Copy link
Contributor

kdid commented Apr 5, 2017

@cpd3149 - #177 fixes batch uploads and is deployed to staging to test. We had overidden some code in Sufia views that changed with the upgrade.

The (!) validation on adding files or descriptions appears to be an issue with Sufia and not specific to us. Note that it is a cosmetic issue (batch still works) so let us know if this is something that is holding up on your signing off pushing the upgrade to production.

@chrisdaaz
Copy link

okay, cool, i've signed off -- thanks @kdid & @bmquinn !

@chrisdaaz
Copy link

@adamjarling @kdid if this moves to prod, will the new fonts and CSS changes move with it? i want to run the css changes by marketing.

@kdid
Copy link
Contributor

kdid commented Apr 7, 2017

@cpd3149 - yes, they would. So just let us know when you get the OK.

This was referenced Apr 7, 2017
@adamjarling
Copy link
Contributor

@cpd3149 Please note the style updates are not complete by any means. Merely a starting point. I estimated with @davidschober there's about 3-5 solid days of work to get it fully presentable.

@kdid
Copy link
Contributor

kdid commented Apr 17, 2017

upgraded on production

@chrisdaaz chrisdaaz mentioned this issue Apr 17, 2017
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants