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(console): show app secret #1723

Merged
merged 1 commit into from
Aug 4, 2022
Merged

Conversation

wangsijie
Copy link
Contributor

Summary

Show app secret in AC for traditional app.

Testing

截屏2022-08-02 下午5 23 24

截屏2022-08-02 下午5 23 28

@linear
Copy link

linear bot commented Aug 2, 2022

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

COMPARE TO master

Total Size Diff 📈 +2.47 KB

Diff by File
Name Diff
packages/console/src/components/CopyToClipboard/index.module.scss 📈 +92 Bytes
packages/console/src/components/CopyToClipboard/index.tsx 📈 +805 Bytes
packages/console/src/icons/EyeClosed.tsx 📈 +1.09 KB
packages/console/src/pages/ApplicationDetails/components/Settings.tsx 📈 +367 Bytes
packages/phrases/src/locales/en/translation/admin-console/application-details.ts 📈 +36 Bytes
packages/phrases/src/locales/ko-kr/translation/admin-console/application-details.ts 📈 +36 Bytes
packages/phrases/src/locales/tr-tr/translation/admin-console/application-details.ts 📈 +42 Bytes
packages/phrases/src/locales/zh-cn/translation/admin-console/application-details.ts 📈 +36 Bytes

@charIeszhao
Copy link
Member

The "Eye" icon should be closed when hiding the secret

@wangsijie
Copy link
Contributor Author

update: add eye close icon

image

Copy link
Contributor

@darcyYe darcyYe left a comment

Choose a reason for hiding this comment

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

LGTM

@wangsijie wangsijie enabled auto-merge (squash) August 4, 2022 05:05
@wangsijie wangsijie merged commit 01dfeed into master Aug 4, 2022
@wangsijie wangsijie deleted the sijie-log-3839-app-secret branch August 4, 2022 05:18
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