Skip to content

feat: added oauth-consent component for oauth authorization - #455

Merged
petruki merged 1 commit into
masterfrom
staging
Oct 4, 2026
Merged

petruki merged 1 commit into
masterfrom
staging

Conversation

@petruki

@petruki petruki commented Oct 4, 2026

Copy link
Copy Markdown
Member

This pull request introduces a complete OAuth authorized applications management feature to the application. It adds a user interface for reviewing, authorizing, and revoking third-party OAuth client access, along with all supporting services, models, and routing. The implementation includes both the OAuth consent screen (shown during authorization) and a new settings page for managing authorized applications.

The most important changes are:

OAuth Consent Flow Implementation:

  • Added the OauthConsentComponent (oauth-consent.component.ts, .html, .css) to display OAuth consent details, requested scopes, and handle user authorization or decline actions. This includes robust validation, error handling, and integration with the backend API. [1] [2] [3]
  • Implemented the OAuthService (oauth.service.ts) for communicating with the backend API to submit consent decisions, fetch authorized apps, and revoke access.
  • Introduced the AuthorizedApp model to represent authorized OAuth clients.

Authorized Apps Management Page:

  • Added the AuthorizedAppsComponent (settings-authorized-apps.component.ts, .html, .css) and integrated it into the settings module. This page lists authorized apps, shows details, and allows revocation of access. It handles loading states, errors, and "coming soon" fallback if the backend is not implemented. [1] [2] [3]
  • Updated the settings navigation and routing to include the new "Authorized Apps" page. [1] [2] [3]

@petruki petruki added this to the v2.2.1 milestone Oct 4, 2026
@petruki petruki self-assigned this Oct 4, 2026
@petruki petruki added enhancement New feature or request ai-generated Indicates that AI tools were used to generate this change labels Oct 4, 2026
@petruki
petruki merged commit f86328b into master Oct 4, 2026
3 checks passed
@petruki
petruki deleted the staging branch October 4, 2026 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Indicates that AI tools were used to generate this change enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant