Open paregorios opened 5 years ago
Blocker for #374 (upgrade site to Plone 5)
I believe that these are the vocabularies that are still using ATVM:
I've created PRs in pleiades.vocabularies
and Products.PleiadesEntities
, merged those branches to staging, deployed and run the migrations. All the vocabs on staging should now be registry based vocabularies, but otherwise be identical to the ATVM vocabularies. This will require relatively thorough testing because these vocabularies are used extensively, including in the exports.
Additionally, once this is approved I plan on making minor changes to a number of other repositories to remove their dependencies on ATVM. Those are all trivial test or package install time dependencies, so the changes should not have any impact on functionality. I have already tested those changes locally, but didn't want to make a dozen more PRs.
Below is the testing regime I propose to follow:
/vocabularies/
The expected effect is:
The following vocabularies include the ability to "hide" terms (based on the affordances in the settings control panel):
@alecpm can you think of any other steps I should be taking to test this rigorously?
I believe that's all that should be involved, but you would definitely know better than I do. There are some scripts like Products/PleiadesEntity/skins/PleiadesEntity/getNameFacets.py
used for Products/PleiadesEntity/skins/PleiadesEntity/browse_names.pt
that make use of the vocabs as well, I'm not sure where those appear on the site though.
It looks like #394 may be related and so I've marked it as blocking progress on this ticket.
Definitely related. I believe a fix is on its way to the staging server right now. I also opened up a new PR on pleiades.policy
.
This should no longer be blocked and is ready for review.
All branches have been merged and deployed to production, and the migration was run.
Evaluation on production: I cannot see an entry for "archaeological remains" on /vocabularies
. Based on the checklist above, I previously saw it on staging after the upgrade, so this looks perhaps like a missed step. I wanted to double-check staging, but it is timing out this morning.
What's odd is the "archaeological remains" was added as a registry based vocabulary 3 years ago. I'm going to try running the old upgrade steps for that vocab to see if they put it in place, but I don't understand why it would have gone missing. I'll also take a look at staging.
Looks like the vocabulary is there, but it's not included in the vocabulary view. You can see it by searching the registry (https://pleiades.stoa.org/portal_registry) for "arch_remains". I'll add the vocab id to the listing view.
It wasn't listed on staging either, but now it's in both. Turns out the vocabs that weren't originally ATVM vocabs need to be hardcoded into the custom listing view to appear there.
Looks good. Closing as resolved.
We never did the last bit, but we should: "Remove ATVocabularyManager from the Pleiades buildout. Hunt down and exterminate related cruft."
Finish removing dependency on ATVocabularyManager and migrate any remaining vocabs to PleiadesSettings (to include surfacing the new vocab in HTML at /vocabularies/ and to include exporting same to JSON).
Remove ATVocabularyManager from the Pleiades buildout. Hunt down and exterminate related cruft.