Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(schemas): add isVisible to applications table #6766

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Nov 4, 2024

Summary

add isVisible to applications table

For each SAML application, we will create a hidden (with isVisible column marked as false) OIDC application to help complete the auth flow because Logto is based on OIDC.

Testing

Should be covered by CI, will add unit tests and integration tests when we implement LOG-10135

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

github-actions bot commented Nov 4, 2024

COMPARE TO master

Total Size Diff 📈 +1.17 KB

Diff by File
Name Diff
packages/core/src/mocks/index.ts 📈 +38 Bytes
packages/schemas/alterations/next-1730712629-add-is-visible-for-applications-table.ts 📈 +884 Bytes
packages/schemas/src/seeds/application.ts 📈 +19 Bytes
packages/schemas/tables/applications.sql 📈 +260 Bytes

@darcyYe darcyYe force-pushed the yemq-log-10133-add-isVisible-to-applications-table branch from c3094a5 to 08039f4 Compare November 4, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant