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

[v15] Always show Bots page with conditional permissions error #50185

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

avatus
Copy link
Contributor

@avatus avatus commented Dec 13, 2024

Backport #50162 to branch/v15

This will make the bots page always visible to users. If they do not
have permissions, they will see the empty state as well as an error of
what permissions are missing.
@avatus avatus added the no-changelog Indicates that a PR does not require a changelog entry label Dec 13, 2024
@avatus avatus enabled auto-merge December 13, 2024 00:59
@github-actions github-actions bot requested review from rudream and zmb3 December 13, 2024 01:00
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-50185.d3b94eevwi10ji.amplifyapp.com

@avatus avatus added this pull request to the merge queue Dec 13, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 13, 2024
@avatus avatus added this pull request to the merge queue Dec 13, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 13, 2024
@avatus avatus added this pull request to the merge queue Dec 13, 2024
Merged via the queue into branch/v15 with commit 72d2084 Dec 13, 2024
36 of 37 checks passed
@avatus avatus deleted the bot/backport-50162-branch/v15 branch December 13, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/sm ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants