chore(app): bump authorizer-react to 2.2.0-rc.7 - #754
Merged
Conversation
Pulls in authorizer-js 4.0.0-rc.0, which went major for the id-only DeleteUserRequest. No behaviour change here: the entire js source diff is 8 lines in admin-only code and the login UI never calls deleteUser.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Completes the SDK release chain for the id-only
DeleteUserchange (#753).The chain
v0.2.0-rc.10.2.0rc14.0.0-rc.0DeleteUserRequest.email→.id2.2.0-rc.7authorizer-js went major because a public type lost a required field, so
^3.3.0in authorizer-react would never have resolved it.Blast radius here is nil, and I checked rather than assumed
v3.3.0..v4.0.0-rc.0source diff is 8 lines acrosssrc/admin.tsandsrc/types.ts— admin-only.web/appnever callsdeleteUser(it is the login UI, not the admin UI).rc.6..rc.7is the dependency bump alone; no source change.Verified
npm installresolves the whole chain: authorizer-react2.2.0-rc.7→ authorizer-js4.0.0-rc.0make build-app— succeedsmake lint-ts— cleanBoth npm packages are published on the
rcdist-tag, withlatestdeliberately untouched (authorizer-js@latestis still3.3.0), so no existing consumer is pulled onto a prerelease.