Skip to content

feat: adds authorization remove command @W-23970421@ - #1280

Merged
tshopshireSalesforce merged 1 commit into
packaging-distribution/spifrom
t/packaging-distribution/authorizeremove
Aug 31, 2026
Merged

feat: adds authorization remove command @W-23970421@#1280
tshopshireSalesforce merged 1 commit into
packaging-distribution/spifrom
t/packaging-distribution/authorizeremove

Conversation

@tshopshireSalesforce

Copy link
Copy Markdown
Collaborator

What does this PR do?

Adds authorization remove command for Limited Packages.

What issues does this PR fix or reference?

@tshopshireSalesforce
tshopshireSalesforce requested a review from a team as a code owner August 28, 2026 19:54
@tshopshireSalesforce
tshopshireSalesforce requested review from btrn11 and sshyamala-sfdc and removed request for a team August 28, 2026 19:54

@jshackell-sfdc jshackell-sfdc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See my suggestion.


# description

Remove one authorization matching the subscriber org ID. Without --package, the command removes the authorization that isn't scoped to a package; specify --package to remove a package-scoped authorization.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Remove one authorization matching the subscriber org ID. Without --package, the command removes the authorization that isn't scoped to a package; specify --package to remove a package-scoped authorization.
Remove one authorization matching the subscriber org ID. Specify --package to remove a package-scoped authorization.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Okay? The double-negative is confusing, so I think it's fine to just leave the positive section. If I've removed something important, please try not to use a double-negative.

@tshopshireSalesforce

Copy link
Copy Markdown
Collaborator Author

See my suggestion.

Hey Juliet, we were planning on having one doc review for all of the messages once we have completed the functional work. That way everything will be cohesive and can be done in one go.

@tshopshireSalesforce
tshopshireSalesforce merged commit fc34d10 into packaging-distribution/spi Aug 31, 2026
14 checks passed
@tshopshireSalesforce
tshopshireSalesforce deleted the t/packaging-distribution/authorizeremove branch August 31, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants