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(core,console,schemas): update variable name and field name #5617

Merged

Conversation

darcyYe
Copy link
Contributor

@darcyYe darcyYe commented Apr 2, 2024

Summary

Rename:

  1. enum type LogtoJwtTokenPath to LogtoJwtTokenKeyType.
  2. envVars field in JWT customizer to be environmentVariables.

Since this is a breaking change, we have to add some manual type casting. Will remove those temporary type casting after the @logto/cloud repo update the dependency and publish new release.

Testing

N/A. Secured by CI.

Checklist

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

Copy link

github-actions bot commented Apr 2, 2024

COMPARE TO master

Total Size Diff 📈 +958 Bytes

Diff by File
Name Diff
packages/console/src/pages/CustomizeJwt/CreateButton/index.tsx 📈 +6 Bytes
packages/console/src/pages/CustomizeJwt/CustomizerItem/index.tsx 📈 +9 Bytes
packages/console/src/pages/CustomizeJwt/index.tsx 📈 +15 Bytes
packages/console/src/pages/CustomizeJwt/utils/path.ts 📈 +9 Bytes
packages/console/src/pages/CustomizeJwtDetails/MainContent/ScriptSection/index.tsx 📈 +6 Bytes
packages/console/src/pages/CustomizeJwtDetails/MainContent/SettingsSection/InstructionTab/index.tsx 📈 +9 Bytes
packages/console/src/pages/CustomizeJwtDetails/MainContent/SettingsSection/TestTab/index.tsx 📈 +6 Bytes
packages/console/src/pages/CustomizeJwtDetails/MainContent/index.tsx 📈 +9 Bytes
packages/console/src/pages/CustomizeJwtDetails/PageLoadingSkeleton/index.tsx 📈 +9 Bytes
packages/console/src/pages/CustomizeJwtDetails/index.tsx 📈 +6 Bytes
packages/console/src/pages/CustomizeJwtDetails/type.ts 📈 +21 Bytes
packages/console/src/pages/CustomizeJwtDetails/use-data-fetch.ts 📈 +6 Bytes
packages/console/src/pages/CustomizeJwtDetails/utils/format.ts 📈 +64 Bytes
packages/console/src/pages/CustomizeJwtDetails/utils/path.ts 📈 +6 Bytes
packages/core/src/mocks/index.ts 📈 +26 Bytes
packages/core/src/oidc/init.ts 📈 +142 Bytes
packages/core/src/routes/logto-config/jwt-customizer.ts 📈 +269 Bytes
packages/core/src/routes/logto-config/logto-config.openapi.json 📈 +39 Bytes
packages/integration-tests/src/mocks/jwt-customizer.ts 📈 +13 Bytes
packages/schemas/src/types/logto-config/jwt-customizer.test.ts 📈 +26 Bytes
packages/schemas/src/types/logto-config/jwt-customizer.ts 📈 +262 Bytes

@darcyYe darcyYe force-pushed the yemq-log-8449-log-8489-update-enum-type-name-and-field-name branch 2 times, most recently from 0902ecc to c4a09c0 Compare April 2, 2024 08:29
@darcyYe darcyYe force-pushed the yemq-log-8449-log-8489-update-enum-type-name-and-field-name branch from c4a09c0 to bde2abe Compare April 3, 2024 03:58
@darcyYe darcyYe enabled auto-merge April 3, 2024 03:59
@darcyYe darcyYe merged commit 8e1713f into master Apr 3, 2024
19 checks passed
@darcyYe darcyYe deleted the yemq-log-8449-log-8489-update-enum-type-name-and-field-name branch April 3, 2024 04:11
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.

3 participants