Skip to content

feat(integrations): allow to rotate client secret of Internal/Public integrations, frontend part #69115

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

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

oioki
Copy link
Member

@oioki oioki commented Apr 17, 2024

To be merged after #69015

image

@oioki oioki requested review from ykamo001, mdtro and a team April 17, 2024 17:38
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 17, 2024
Copy link

codecov bot commented Apr 17, 2024

Bundle Report

Changes will increase total bundle size by 1.7kB ⬆️

Bundle name Size Change
app-webpack-bundle-array-push 26.22MB 1.7kB ⬆️

Comment on lines +329 to +340
<Fragment>
<Header>{t('Rotated Client Secret')}</Header>
<Body>
<Alert type="info" showIcon>
{t('This will be the only time your client secret is visible!')}
</Alert>
<p>
{t('Your client secret is:')}
<code>{rotateResponse.clientSecret}</code>
</p>
</Body>
</Fragment>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think i created a component such that the value is only shown once, would that fit your needs? You wouldn't have to worry about wording and everything because you could just pass in the token

@oioki oioki enabled auto-merge (squash) April 19, 2024 07:59
@oioki oioki merged commit 24d6dce into master Apr 19, 2024
41 checks passed
@oioki oioki deleted the feat/sentry-app-rotate-secret-frontend branch April 19, 2024 08:04
MichaelSun48 pushed a commit that referenced this pull request Apr 25, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants