-
Notifications
You must be signed in to change notification settings - Fork 35
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: testnet mode approve sheet #1083
Conversation
e4a6b42
to
533ceab
Compare
9632258
to
7e5dbd4
Compare
1ea708e
to
4586fce
Compare
Here's the packed extension for this build: |
4586fce
to
a878ae1
Compare
5feb4b7
to
ca631ea
Compare
a878ae1
to
542c649
Compare
Here's the packed extension for this build: |
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.
This is a really nice ux 🤝
|
||
const [hint, setHint] = useState<Hint>(INITIAL_HINT); | ||
|
||
console.log('pendingRequest', pendingRequest); |
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.
🧹 🧹 🧹
ca631ea
to
40d8b95
Compare
542c649
to
5f94b4e
Compare
40d8b95
to
b30201a
Compare
Here's the packed extension for this build: |
5f94b4e
to
6453b83
Compare
6453b83
to
86ef36b
Compare
Here's the packed extension for this build: |
Here's the packed extension for this build: |
dcedc78
to
97575b0
Compare
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.
👍, small comments
size="10pt" | ||
weight="bold" | ||
> | ||
🕹️{' '} |
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.
guessing the formatter did this, or do you want the space here?
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.
i want this
🕹️{' '} | ||
<TextLink scale={false} color="green"> | ||
{i18n.t('testnet_mode_watcher.testnet_mode')} | ||
</TextLink>{' '} |
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.
same here
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.
but not this, will fix thanks
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.
actually, i do want it, is a link inside a text
Here's the packed extension for this build: |
Here's the packed extension for this build: |
Here's the packed extension for this build: |
Here's the packed extension for this build: |
567d33a
to
8cd812b
Compare
Here's the packed extension for this build: |
Here's the packed extension for this build: |
Fixes BX-####
Figma link (if any):
What changed (plus any additional context for devs)
we're now showing a sheet making the users
Screen recordings / screenshots
https://www.loom.com/share/16f1f9f047c54316bf5409e56db8291d
What to test