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

chore: add SAML app API docs #6967

Open
wants to merge 5 commits into
base: yemq-remove-saml-app-feature-guard
Choose a base branch
from

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Jan 22, 2025

Summary

Add SAML app openapi.json file which can help to render API docs. (resolves LOG-10720)

Testing

Covered by CI.

Checklist

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

Copy link

github-actions bot commented Jan 22, 2025

COMPARE TO master

Total Size Diff ⚠️ 📈 +16.61 KB

Diff by File
Name Diff
packages/console/src/components/Guide/hooks.ts 📈 +98 Bytes
packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/AttributeMapping.tsx 📈 +45 Bytes
packages/core/src/routes/applications/application.ts 📈 +98 Bytes
packages/core/src/routes/init.ts 📈 +342 Bytes
packages/core/src/routes/saml-application/anonymous.ts 📈 +1.5 KB
packages/core/src/routes/saml-application/index.openapi.json 📈 +9.4 KB
packages/core/src/routes/saml-application/saml-app-auth-flow.openapi.json 📈 +5.65 KB
packages/core/src/routes/swagger/utils/documents.ts 📈 +716 Bytes
packages/core/src/routes/swagger/utils/general.ts 📈 +182 Bytes
packages/core/src/routes/swagger/utils/operation.ts 📈 +338 Bytes

@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch 2 times, most recently from d175ba8 to b48db3f Compare January 22, 2025 04:54
Copy link
Contributor

@wangsijie wangsijie left a comment

Choose a reason for hiding this comment

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

Remember to resolve the failed workflows.

Base automatically changed from yemq-reorg-saml-app-code to master January 22, 2025 07:42
@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch 2 times, most recently from b6e166e to 5c81864 Compare January 22, 2025 17:44
@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch 3 times, most recently from 83cf58a to 1b4c537 Compare January 26, 2025 13:41
@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch from 1b4c537 to c041b05 Compare January 26, 2025 14:50
@darcyYe darcyYe changed the base branch from master to yemq-remove-saml-app-feature-guard January 26, 2025 14:51
@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch from c041b05 to 5970c0a Compare January 26, 2025 15:17
@darcyYe darcyYe force-pushed the yemq-add-saml-app-api-docs branch from 7314107 to f7d8dfa Compare January 26, 2025 16:43
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.

2 participants