Skip to content

Redesign OAuth2 authorize screen, #PG-4966#15

Merged
AltamashShaikh merged 4 commits into5.x-devfrom
PG-4966-auth-screen-design
Apr 6, 2026
Merged

Redesign OAuth2 authorize screen, #PG-4966#15
AltamashShaikh merged 4 commits into5.x-devfrom
PG-4966-auth-screen-design

Conversation

@AltamashShaikh
Copy link
Copy Markdown
Contributor

@AltamashShaikh AltamashShaikh commented Apr 1, 2026

Description

Redesign OAuth2 authorize screen, #PG-4966

Issue No

#PG-4966

Steps to Replicate the Issue

  1. View the auth screen to see the new design

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)
  • [NA] Version bumped?
  • [NA] I have understood, reviewed, and tested all AI outputs before use
  • [NA] All AI instructions respect security, IP, and privacy rules

@AltamashShaikh AltamashShaikh requested a review from lachiebol April 2, 2026 00:54
@AltamashShaikh AltamashShaikh added the Needs Review For pull requests that need a code review. label Apr 2, 2026
Copy link
Copy Markdown
Contributor

@lachiebol lachiebol left a comment

Choose a reason for hiding this comment

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

Few small changes, functionally works well 😄

Comment thread stylesheets/ouath2.less Outdated
Comment thread lang/en.json Outdated
Comment thread OAuth2.php Outdated
@AltamashShaikh AltamashShaikh requested a review from lachiebol April 2, 2026 04:25
@AltamashShaikh
Copy link
Copy Markdown
Contributor Author

Will wait for Valeriia to confirm and merge it accordingly.

@AltamashShaikh AltamashShaikh merged commit 1a445a6 into 5.x-dev Apr 6, 2026
8 checks passed
@AltamashShaikh AltamashShaikh deleted the PG-4966-auth-screen-design branch April 6, 2026 01:31
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