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

Review Sheet #5632

Merged
merged 11 commits into from Apr 19, 2024
Merged

Review Sheet #5632

merged 11 commits into from Apr 19, 2024

Conversation

walmat
Copy link
Contributor

@walmat walmat commented Apr 16, 2024

Fixes APP-1302, APP-1361

What changed (plus any additional context for devs)

Added review panel that will pop-up when the user clicks the review button. This PR doesn't include the logic to make the button disabled or whatever before the proper logic is in place to prompt for review.

Follow-up PRs:
Hook up fee to review panel
Hook up gas configuration to review panel
Clean up animation transitions

Copy link

linear bot commented Apr 16, 2024

APP-1302 Review Sheet

@walmat walmat changed the title initial setup for review sheet Review Sheet Apr 16, 2024
@walmat walmat marked this pull request as ready for review April 17, 2024 15:36
Copy link

linear bot commented Apr 18, 2024

Copy link
Contributor

@skylarbarrera skylarbarrera left a comment

Choose a reason for hiding this comment

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

homies in the lab

@walmat walmat merged commit ed8e69c into develop Apr 19, 2024
6 checks passed
@walmat walmat deleted the @matthew/APP-1302 branch April 19, 2024 13:39
BrodyHughes added a commit that referenced this pull request Apr 22, 2024
…eplink-add

* 'develop' of github.com:rainbow-me/rainbow:
  Cover a few more edge cases with showing the swap warnings (#5648)
  refetch quote if a user selects a new assset to sell with a buy asset already selected (#5643)
  Fix random quote edge cases (#5644)
  use dapp metadata where possible (#5629)
  Output chain selection context menu (#5630)
  chore: use native currency balances (#5631)
  browser: android fixes (#5642)
  Review Sheet (#5632)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants