-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
@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. |
@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. |
currently testing. it seems to be working, overall, but there's a weird span element
displaying between the Title and Abstract: https://nufiaweb-s.library.northwestern.edu/concern/generic_works/sq87bt648 this appears on new works since the upgrade: 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. i don't know if this related to the upgrade. everything else looks good to me! |
Yes, batch upload not working. Looking into it. |
Here is an issue related to "deposited" showing. |
@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. |
@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. |
@cpd3149 - yes, they would. So just let us know when you get the OK. |
@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. |
upgraded on production |
done looks like
The text was updated successfully, but these errors were encountered: