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

6.2 release notes #10432

Merged
merged 67 commits into from
Apr 1, 2024
Merged

6.2 release notes #10432

merged 67 commits into from
Apr 1, 2024

Conversation

jp-tosca
Copy link
Contributor

What this PR does / why we need it:

Dataverse 6.2 release notes.

Which issue(s) this PR closes:

Closes #10422

@jp-tosca jp-tosca added the Size: 10 A percentage of a sprint. 7 hours. label Mar 26, 2024
@jp-tosca jp-tosca self-assigned this Mar 26, 2024
Copy link
Member

@pdurbin pdurbin left a 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.

doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Show resolved Hide resolved
doc/release-notes/6.2-release-notes.md Outdated Show resolved Hide resolved
@scolapasta scolapasta added this to the 6.2 milestone Mar 27, 2024
@coveralls
Copy link

Coverage Status

coverage: 20.736%. remained the same
when pulling e00f630 on 10422-62-release-notes
into d5bb0c2 on develop.

@jp-tosca jp-tosca marked this pull request as ready for review March 28, 2024 19:50
@jp-tosca jp-tosca requested a review from pdurbin March 28, 2024 19:58
Copy link
Member

@pdurbin pdurbin left a 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! 🚀 🎉

@sekmiller sekmiller self-assigned this Mar 29, 2024
@landreev
Copy link
Contributor

@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?

@landreev
Copy link
Contributor

@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.
Should we add something like "we recommend that you update your metadata blocks just in case" in one or both places? And maybe "if a block update fails because of duplicate values, see the paragraph earlier in the release note on how to resolve it" in the upgrade instructions? - assuming I actually understand what's going on there.

(this is all small scale/nitpick-level stuff. so, up to you).

@qqmyers
Copy link
Member

qqmyers commented Mar 29, 2024

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.

@landreev
Copy link
Contributor

OK, I just looked at #9983 and it focused on the missing constraints, so I assumed #9984 was just to address that.
I would maybe just say "Update the following blocks to reflect the ongoing improvements made to the handling of core metadata fields" then - unless you can suggest something better.

@sekmiller sekmiller removed their assignment Mar 29, 2024
@pdurbin pdurbin merged commit eade48c into develop Apr 1, 2024
1 of 2 checks passed
@pdurbin pdurbin deleted the 10422-62-release-notes branch April 1, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: 10 A percentage of a sprint. 7 hours.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6.2: Create release notes
8 participants