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

wallet-ext: hide max stake button #7677

Merged
merged 1 commit into from
Jan 25, 2023
Merged

Conversation

pchrysochoidis
Copy link
Contributor

  • hide the max stake button for now since most of the time using the max stake amount that it sets causes the stake transaction to fail
before after
Screenshot 2023-01-25 at 16 17 41 Screenshot 2023-01-25 at 16 17 02

part of: APPS-387

@vercel
Copy link

vercel bot commented Jan 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Ignored Deployments
Name Status Preview Comments Updated
explorer ⬜️ Ignored (Inspect) Jan 25, 2023 at 4:52PM (UTC)
explorer-storybook ⬜️ Ignored (Inspect) Jan 25, 2023 at 4:52PM (UTC)
frenemies ⬜️ Ignored (Inspect) Jan 25, 2023 at 4:52PM (UTC)
wallet-adapter ⬜️ Ignored (Inspect) Jan 25, 2023 at 4:52PM (UTC)

@pchrysochoidis pchrysochoidis merged commit 1224cd5 into main Jan 25, 2023
@pchrysochoidis pchrysochoidis deleted the pc-wallet-ext-apps-387 branch January 25, 2023 17:00
williampsmith pushed a commit that referenced this pull request Feb 3, 2023
* hide the max stake button for now since most of the time using the max
stake amount that it sets causes the stake transaction to fail

| before | after |
| - | - |
| <img width="394" alt="Screenshot 2023-01-25 at 16 17 41"
src="https://user-images.githubusercontent.com/10210143/214617037-fd643aca-bf52-4dbb-bd24-2cc0aac855ae.png">
| <img width="394" alt="Screenshot 2023-01-25 at 16 17 02"
src="https://user-images.githubusercontent.com/10210143/214617102-16357e12-e64b-4283-b743-ae3630228026.png">
|

part of: APPS-387
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants