Skip to content

Adds code to warn users, when scope is downgraded - #42

Merged
AltamashShaikh merged 1 commit into
5.x-devfrom
warn-on-access-downgrade
Jul 23, 2026
Merged

Adds code to warn users, when scope is downgraded#42
AltamashShaikh merged 1 commit into
5.x-devfrom
warn-on-access-downgrade

Conversation

@AltamashShaikh

Copy link
Copy Markdown
Contributor

Description

Adds code to wanr users, when scope is downgraded

Issue No

Steps to Replicate the Issue

  1. Edit an existing OAuth client with admin access
  2. Change the scope to read and notice no notification on downgrade informing exiting token will keep working till the TTL
  3. Checkout this PR and perform same step and now it should show a warning

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [✔] New test case added/updated?
  • [✔] Are all newly added texts included via translation?
  • [✔] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✔] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

@AltamashShaikh AltamashShaikh changed the title Adds code to wanr users, when scope is downgraded Adds code to warn users, when scope is downgraded Jul 22, 2026
@AltamashShaikh AltamashShaikh added the Needs Review For pull requests that need a code review. label Jul 22, 2026
@AltamashShaikh
AltamashShaikh requested a review from a team July 22, 2026 08:01
@AltamashShaikh
AltamashShaikh merged commit 999437e into 5.x-dev Jul 23, 2026
10 checks passed
@AltamashShaikh
AltamashShaikh deleted the warn-on-access-downgrade branch July 23, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review For pull requests that need a code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants