-
Notifications
You must be signed in to change notification settings - Fork 25
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
fix(ui): permissions on intro & updated settings with anon id #1128
Conversation
<SettingsGroupContent> | ||
<Stack direction="row" alignItems="center" justifyContent="flex-start"> | ||
{/* eslint-disable-next-line i18next/no-literal-string */} | ||
<Typography variant="p">Anon ID: {anon_id}</Typography> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
disabled string check because anon_id is a field name we don't translate, but LMK if that should also be translated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anon id looks good. Copys fine.
Telemetry toggle stays on the splashscreen on every start despite the new app config option for creation date. Tested with existing wallet, and fresh install.
Description
created_at
field to app_config to check if it's the first ever run of the app or not (to hide permissions toggle on the Intro screens on subsequent runs)anon_id
which can be copiedMotivation and Context
resolves:
How Has This Been Tested?
Screen.Recording.2024-11-26.at.16.mp4
What process can a PR reviewer use to test or verify this change?
re. #1123:
re. #1114: