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(core): add PATCH/GET /saml-applications/:id APIs #6827

Open
wants to merge 1 commit into
base: yemq-add-post-saml-app-api
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Nov 22, 2024

Summary

Add following APIs:

  1. PATCH /saml-applications/:id
  2. GET /saml-applications/:id

This PR resolves LOG-10115 and LOG-10116

Testing

Add integration tests.

Checklist

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

@darcyYe darcyYe requested a review from a team November 22, 2024 10:23
@github-actions github-actions bot added the feature Cool stuff label Nov 22, 2024
Copy link

COMPARE TO master

Total Size Diff 📈 +5.8 KB

Diff by File
Name Diff
.changeset/early-numbers-attend.md 📈 +1.53 KB
.changeset/four-deers-divide.md 📈 +694 Bytes
.changeset/warm-pugs-divide.md 📈 +1.87 KB
.changeset/wet-nails-guess.md 📈 +855 Bytes
packages/cli/CHANGELOG.md 📈 +193 Bytes
packages/cli/package.json 0 Bytes
packages/console/CHANGELOG.md 📈 +2.77 KB
packages/console/package.json 0 Bytes
packages/core/CHANGELOG.md 📈 +4.93 KB
packages/core/package.json 📈 +182 Bytes
packages/core/src/routes/applications/application.ts 📈 +15 Bytes
packages/core/src/routes/init.ts 📈 +177 Bytes
packages/core/src/routes/swagger/utils/documents.ts 📈 +100 Bytes
packages/core/src/routes/swagger/utils/general.ts 📈 +1017 Bytes
packages/core/src/routes/well-known/well-known.openapi.ts 📈 +1 Bytes
packages/core/src/saml-applications/libraries/saml-applications.ts 📈 +3.06 KB
packages/core/src/saml-applications/libraries/utils.ts 📈 +2.34 KB
packages/core/src/saml-applications/queries/configs.ts 📈 +1.14 KB
packages/core/src/saml-applications/queries/secrets.ts 📈 +1.31 KB
packages/core/src/saml-applications/routes/index.openapi.json 📈 +3.2 KB
packages/core/src/saml-applications/routes/index.ts 📈 +3.98 KB
packages/core/src/tenants/Libraries.ts 📈 +169 Bytes
packages/core/src/tenants/Queries.ts 📈 +342 Bytes
packages/create/CHANGELOG.md 📈 +51 Bytes
packages/create/package.json 0 Bytes
packages/experience-legacy/CHANGELOG.md 📈 +544 Bytes
packages/experience-legacy/package.json 📈 +1 Bytes
packages/experience/CHANGELOG.md 📈 +544 Bytes
packages/experience/package.json 📈 +1 Bytes
packages/integration-tests/CHANGELOG.md 📈 +1.81 KB
packages/integration-tests/package.json 📈 +1 Bytes
packages/integration-tests/src/api/saml-application.ts 📈 +878 Bytes
packages/integration-tests/src/tests/api/application/application.test.ts 📈 +6 Bytes
packages/integration-tests/src/tests/api/application/saml-application.test.ts 📈 +3.62 KB
packages/phrases-experience/CHANGELOG.md 📈 +543 Bytes
packages/phrases-experience/package.json 0 Bytes
packages/phrases/CHANGELOG.md 📈 +4.55 KB
packages/phrases/package.json 0 Bytes
packages/phrases/src/locales/en/errors/application.ts 📈 +207 Bytes
packages/schemas/CHANGELOG.md 📈 +3.32 KB
packages/schemas/alterations/1.22.0-1730689363-add-account-center.ts 📈 +965 Bytes
packages/schemas/alterations/1.22.0-1731054001-init-account-center.ts 📈 +1.22 KB
packages/schemas/alterations/1.22.0-1731304920-add-support-email-and-website-to-sie-table.ts 📈 +546 Bytes
packages/schemas/alterations/1.22.0-1731377260-add-unknown-session-redirect-url-to-sie.ts 📈 +490 Bytes
packages/schemas/alterations/1.22.0-1731900596-add-saml-application-type.ts 📈 +1.88 KB
packages/schemas/alterations/1.22.0-1731900631-add-saml-app-third-party-consistency-check.ts 📈 +569 Bytes
packages/schemas/alterations/1.22.0-1731901231-add-saml-application-secrets-table.ts 📈 +1.35 KB
packages/schemas/alterations/1.22.0-1731904029-add-saml-application-configs-table.ts 📈 +1.13 KB
packages/schemas/alterations/next-1730689363-add-account-center.ts 📈 +965 Bytes
packages/schemas/alterations/next-1731054001-init-account-center.ts 📈 +1.22 KB
packages/schemas/alterations/next-1731304920-add-support-email-and-website-to-sie-table.ts 📈 +546 Bytes
packages/schemas/alterations/next-1731377260-add-unknown-session-redirect-url-to-sie.ts 📈 +490 Bytes
packages/schemas/alterations/next-1731900596-add-saml-application-type.ts 📈 +1.88 KB
packages/schemas/alterations/next-1731900631-add-saml-app-third-party-consistency-check.ts 📈 +569 Bytes
packages/schemas/alterations/next-1731901231-add-saml-application-secrets-table.ts 📈 +1.35 KB
packages/schemas/alterations/next-1731904029-add-saml-application-configs-table.ts 📈 +1.13 KB
packages/schemas/package.json 0 Bytes
packages/schemas/src/types/index.ts 📈 +39 Bytes
packages/schemas/src/types/saml-application.ts 📈 +2.09 KB
packages/schemas/tables/saml_application_configs.sql 📈 +69 Bytes
packages/toolkit/connector-kit/CHANGELOG.md 📈 +1.81 KB
packages/toolkit/connector-kit/package.json 0 Bytes
pnpm-lock.yaml 📈 +1.95 KB

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