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

adding 6.1 release notes and removing .md files #10166

Merged
merged 30 commits into from
Dec 12, 2023

Conversation

stevenwinship
Copy link
Contributor

What this PR does / why we need it: Supplies release notes for v 6.1

Which issue(s) this PR closes:#10151

Closes ##10151

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

@stevenwinship stevenwinship linked an issue Dec 4, 2023 that may be closed by this pull request
@stevenwinship stevenwinship marked this pull request as ready for review December 4, 2023 18:28
@scolapasta scolapasta added this to the 6.1 milestone Dec 4, 2023
@stevenwinship stevenwinship marked this pull request as draft December 4, 2023 20:00
@pdurbin pdurbin self-assigned this Dec 6, 2023
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.

Good first run at this. I took a quick look and left comments.

doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved

## Detailed Release Highlights, New Features and Use Case Scenarios

### Dataverse installation can be now be configured to allow out-of-band upload
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This heading is the first one without Title Case. Let's be consistent. We can ditch title case everywhere or use it everywhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this still an issue. I would like to get the formatting right so we can make a template for future release notes?

doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved
doc/release-notes/6.1-release-notes.md Outdated Show resolved Hide resolved

Upgrading requires a maintenance window and downtime. Please plan ahead, create backups of your database, etc.

These instructions assume that you've already upgraded through all the 5.x releases and are now running Dataverse 6.0.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where are the upgrade instructions?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't see any upgrade instructions. If there are none I will remove this

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sekmiller is working on this ❤️

doc/release-notes/6.1-release-notes.md Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Dec 6, 2023

Coverage Status

coverage: 20.176%. remained the same
when pulling 011b929 on 10151-61-create-release-notes
into b2c37af on develop.

stevenwinship and others added 2 commits December 6, 2023 11:01
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
- hasBeenDeleted (/api/files/{id}/hasBeenDeleted): Know if a particular file that existed in a previous version of the dataset no longer exists in the latest version.
- deaccessionDataset (/api/datasets/{id}/versions/{versionId}/deaccession): version deaccessioning through API (Given a dataset and a version).
- getZipDownloadLimit (/api/info/zipDownloadLimit): Get the configured zip file download limit. The response contains the long value of the limit in bytes.
- getMaxEmbargoDurationInMonths (/api/info/settings/:MaxEmbargoDurationInMonths): Get the maximum embargo duration in months, if available, configured through the database setting :MaxEmbargoDurationInMonths.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this going away with the new settings API?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hate removing APIs. If we do, it should go in the API changelog. Perhaps it would be a good idea to indicate that we plan to replace this API with that more general solution, probably in 6.2.

- when files are requested to be included, some database lookup optimizations have been added to improve the performance on datasets with large numbers of files.

This is reflected in the [Dataset Versions API](https://guides.dataverse.org/en/9763-lookup-optimizations/api/native-api.html#dataset-versions-api) section of the Guide.
### The following API endpoints have been added:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are all of these SPA-related? I'm wondering if pointing out that these are all for existing functionality that is now being exposed via API versus new functionality would help (anyone not using the API might skip this section, and it would advertise SPA progress a bit).

- Functionality has been added to help validate dataset JSON prior to dataset creation. There are two new API endpoints in this release. The first takes in a collection alias and returns a custom dataset schema based on the required fields of the collection. The second takes in a collection alias and a dataset JSON file and does an automated validation of the JSON file against the custom schema for the collection. In this release funtionality is limited to json format validation and validating required elements. Future releases will address field types, controlled vocabulary, etc. (Issue #9464 and #9465)

### Solr Improvements
- As of this release application-side support is added for the "circuit breaker" mechanism in Solr that makes it drop requests more gracefully when the search engine is experiencing load issues.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There were other performance improvements as well right? (Were there non-Solr performance improvements in this release too?) Would be good to point it out.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What issue or PR do you mean?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The performance improvements I can think of are the enhanced /versions apis. But those are mentioned already.

stevenwinship and others added 4 commits December 6, 2023 12:43
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
@scolapasta scolapasta added the Size: 10 A percentage of a sprint. 7 hours. label Dec 6, 2023
@sekmiller sekmiller self-assigned this Dec 7, 2023
@landreev
Copy link
Contributor

landreev commented Dec 7, 2023

I just want to put it on record that there is a long release note in the Globus PR #10162 that I'm still looking into. And there's otherwise a lot of good stuff in that PR and it should definitely be mentioned in the release note.

Comment on lines +137 to +138
- We have started maintaining an API changelog of breaking changes: https://guides.dataverse.org/en/6.1/api/changelog.html
See also #10060.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just pushed an update to make the API changelog only about breaking changes. This will be much easier to maintain. I also called it experimental and said we'd like feedback. Here's how it looks as of this writing:

Screenshot 2023-12-08 at 4 00 44 PM

@sekmiller sekmiller removed their assignment Dec 11, 2023
@sekmiller sekmiller self-assigned this Dec 11, 2023
#### Using MicroProfile Config for provisioning

With this release it is possible to provision a single OIDC-based authentication provider
by using MicroProfile Config instead of or in addition to the classic Admin API provisioning.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you look at the raw file in NetBeans, in most cases we're letting the text run off the page without any line breaks. Does it matter? Do we need to make sure it's consistent?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't matter for the final HTML output.


#### Adding PKCE Support

[This PR adds PKCE support for OIDC providers](https://github.com/IQSS/dataverse/pull/9273)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need these square brackets?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ended up removing this whole line and linking to the guides and that PR lower down.

@sekmiller sekmiller removed their assignment Dec 11, 2023
@pdurbin pdurbin marked this pull request as ready for review December 11, 2023 20:43
@scolapasta scolapasta merged commit 3937e5f into develop Dec 12, 2023
3 checks passed
@scolapasta scolapasta deleted the 10151-61-create-release-notes branch December 12, 2023 18:17
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.1: Create release notes
7 participants