UILD-854: Release v2.0.5 - #441
Conversation
* Expand from one saved setting per profile to multiple saved settings per profile * Update manage settings feature to accommodate creating new settings; update existing tests to run correctly * Merge * Add new tests for profile settings API and modal state dispatch * Add tests for profile settings list component * Add test for settings name and choosing settings from profile selection modal * Add tests for save profile settings hook * Add tests for hooks and default ProfileSettings case * Add tests for loading meta hook and reset components button * Update settings selector to apply settings and persist user values without going through save * Disable selected profile settings * Add tests for profile setting selector component
* Make sure resets are placed properly. Add accessor for resource type URI to allow properly loading authority profiles in settings manager. * Upgrade to @types/node 26.1.1 and add required ambient globals to tsconfig's types list; this is no longer automatically filled with everything in @types to avoid bloated builds * generateRecord only generates enough for the save API, but to properly edit requires retaining reference data without overwriting bfids in stores; add a method to merge the generated record with the existing record's ref, and only apply to certain stores instead of the record and bfids
|
zburke
left a comment
There was a problem hiding this comment.
A patch release with changes to nearly 400 files is surprising and, frankly, terrifying. Rather than a patch-release-specific branch, I expected the target branch here to be b2.0, where the v2.0.4 release would be the most-recent commit before these cherry-picks.
@zepheiryan, what kind of review are you hoping for on a release commit? Given the original PRs have already been approved, is it necessary to seek approval again?
@zburke My prior Citation colleagues were responsible for previous releases, this is the first one I've taken on. Is this part of the process - just before tagging - not usually conditioned on approval (of the prep, not the already-approved code)? The plan from here was to tag the newly merged branch |
|
@zepheiryan, yes, I think release-related commits have been handled internal to teams in the past. There are likely a lot of "because that's how we've always done it" practices, rather than "best" practices, here. I am open to suggestions for improvement. |
@zburke Thanks, I'll close this out and just make the release. I'll take the process as Chesterton's fence for now. |



Purpose
Prepare v2.0.5 release.
Approach
Cherry pick all commits related to UILD-744, UILD-816, UILD-780
Refs
https://folio-org.atlassian.net/browse/UILD-854