Skip to content

Add OISY to direct OpenID flow attributes allowlist#3654

Merged
sea-snake merged 1 commit intomainfrom
sea-snake/add-oisy-to-attributes-allowlist
Mar 6, 2026
Merged

Add OISY to direct OpenID flow attributes allowlist#3654
sea-snake merged 1 commit intomainfrom
sea-snake/add-oisy-to-attributes-allowlist

Conversation

@sea-snake
Copy link
Contributor

Add OISY to direct OpenID flow attributes allowlist

Changes

  • Add certified_attributes: true to OISY entry.
  • Remove rounded corners from OISY logo to make it work with latest redirect UI.

Copy link
Contributor

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

This PR updates the dapps explorer metadata to allow OISY Wallet to use certified attributes in the direct OpenID flow, and adjusts the OISY logo asset to match the latest redirect UI styling.

Changes:

  • Added certified_attributes: true to the OISY Wallet entry in the dapps allowlist.
  • Updated oisy_logo.svg to remove rounded corners (by removing the rx on the background rect).

Reviewed changes

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

File Description
src/frontend/src/lib/legacy/flows/dappsExplorer/dapps.json Enables certified attributes for OISY by setting certified_attributes: true on its dapp descriptor.
src/frontend/src/lib/icons/oisy_logo.svg Removes the rounded-corner background to better fit the redirect UI’s image masking/styling.

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

@sea-snake sea-snake enabled auto-merge March 6, 2026 13:23
@sea-snake sea-snake added this pull request to the merge queue Mar 6, 2026
Merged via the queue into main with commit 9546cf5 Mar 6, 2026
70 checks passed
@sea-snake sea-snake deleted the sea-snake/add-oisy-to-attributes-allowlist branch March 6, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants