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: Add new App Installation Parameter value type 'Secret' [EXT-4836] #2286

Merged

Conversation

jjolton-contentful
Copy link
Contributor

@jjolton-contentful jjolton-contentful commented May 21, 2024

Summary

When creating App Installation Parameter schemas by modifying an App Definition, users can now specify that a parameter is of type "Secret". At the time of this PR this functionality is currently only supported on the schema side.

Description

Motivation and Context

Apps will soon be able to use secret app installation parameters for storing params whose exposure should be limited.

Checklist (check all before merging)

  • Both unit and integration tests are passing
  • There are no breaking changes
  • Changes are reflected in the documentation

Documentation updates to come as this feature matures.

@jjolton-contentful jjolton-contentful requested a review from a team May 21, 2024 21:17
@jjolton-contentful jjolton-contentful force-pushed the feat/secret-app-installation-parameter-type branch from 1d4e8ed to 6a0e24d Compare May 21, 2024 21:44
@jjolton-contentful jjolton-contentful marked this pull request as ready for review May 22, 2024 16:48
@jjolton-contentful jjolton-contentful merged commit c14280f into master May 22, 2024
3 checks passed
@jjolton-contentful jjolton-contentful deleted the feat/secret-app-installation-parameter-type branch May 22, 2024 17:19
@contentful-automation
Copy link
Contributor

🎉 This PR is included in version 11.26.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants