-
Notifications
You must be signed in to change notification settings - Fork 493
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
6.2 release notes #10432
6.2 release notes #10432
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took a quick look.
Co-authored-by: Philip Durbin <[email protected]>
changes to the upgrade instructions order, because of the breaking change in solr schema.
Update 6.2-release-notes.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm exhausted from reading these extremely through release notes over and over. Good enough! Approved! Thanks, @jp-tosca! Nice emojis! 🚀 🎉
@jp-tosca Thank you for making the retroactive fixes to the 6.1 release note. Did you only fix the 2 things you specifically noticed, or did you copy-and-paste the complete source from the release page and back into the .md file on github? |
@qqmyers the upgrade instruction is just saying "update these 4 metadata blocks:" now. We usually say why we are asking people to do that. Is that to resolve any potential constraint issues? Earlier in the note we seem to be going out of our way telling people that it's extremely unlikely that it's a problem in their database. (this is all small scale/nitpick-level stuff. so, up to you). |
I think all 4 of those blocks have updates so would need to be updated, at least if used. I think the astro and bio ones just have the correction to not mark fields as controlled values when there's no associated vocab, but I think they need to be corrected to display properly given the code changes that were made. We could mention that errors in the blocks where fields were marked as controlled vocabulary without having any vocabulary values have been fixed. |
What this PR does / why we need it:
Dataverse 6.2 release notes.
Which issue(s) this PR closes:
Closes #10422