Skip to content

Remove mentions of outdated VIEW_BADGES permission#25

Merged
nscuro merged 1 commit intomainfrom
rm-view-badges
Apr 25, 2026
Merged

Remove mentions of outdated VIEW_BADGES permission#25
nscuro merged 1 commit intomainfrom
rm-view-badges

Conversation

@nscuro
Copy link
Copy Markdown
Member

@nscuro nscuro commented Apr 25, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 25, 2026 21:37
@nscuro nscuro added the defect Something isn't working label Apr 25, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the documentation to remove references to the deprecated VIEW_BADGES permission and the associated “Badge Viewers” team, and simplifies badge URL examples accordingly.

Changes:

  • Removed VIEW_BADGES from the permissions reference and renamed the “Tags and badges” section to “Tags”.
  • Removed the “Badge Viewers” default team entry.
  • Updated badge endpoint examples to no longer include ?apiKey=... and removed the “Access configuration / required permission” guidance section.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/reference/permissions.md Removes VIEW_BADGES and the “Badge Viewers” team from the permissions/default-teams docs.
docs/reference/badges.md Updates badge URL examples and removes badge authentication/permission guidance.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/reference/permissions.md
Comment thread docs/reference/badges.md
Signed-off-by: nscuro <nscuro@protonmail.com>
@nscuro nscuro merged commit 39526b5 into main Apr 25, 2026
5 checks passed
@nscuro nscuro deleted the rm-view-badges branch April 25, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

defect Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants