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.

Sign up for GitHub

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

Jump to bottom

add terra classic bridge warning + refactor #234

Merged
merged 17 commits into from
Jul 13, 2023
Merged

add terra classic bridge warning + refactor #234

merged 17 commits into from
Jul 13, 2023

Conversation

sebastianscatularo
Copy link
Collaborator

@sebastianscatularo sebastianscatularo commented Jul 12, 2023 •
edited
Loading

  • refactor warning message logic to unified
  • limit transfers for terra classic native tokens at steps 1 (Source) and 3 (Redeem)

@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 02:47 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 02:47 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 02:47 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 02:47 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 02:58 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 02:58 — with GitHub Actions Inactive
M-Picco
M-Picco reviewed Jul 12, 2023
View reviewed changes
Copy link
Collaborator

@M-Picco M-Picco left a comment

Choose a reason for hiding this comment

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

Left a few comments.

Nice step into consolidating the restriction checks logic! 👏

src/TransferRules.ts Outdated Show resolved Hide resolved
src/hooks/useRedeemControl.ts Outdated Show resolved Hide resolved
src/components/Transfer/Source.tsx Show resolved Hide resolved
src/hooks/useRedeemControl.ts Show resolved Hide resolved
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 12:35 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 12:35 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 12:35 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 12:44 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 12:44 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 12:44 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 12:44 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 12:53 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 12:55 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo self-assigned this Jul 12, 2023
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 20:57 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 21:07 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 21:10 — with GitHub Actions Inactive
@sebastianscatularo
Copy link
Collaborator Author

sebastianscatularo commented Jul 12, 2023 •
edited
Loading

Pandle Warning and Lock

image Screenshot 2023-07-12 at 18 15 09

Aurora testnet warning

Screenshot 2023-07-12 at 17 35 28 image

Terra Classic Native transfer paused

image Screenshot 2023-07-12 at 17 40 45

Terra Classic Redeem native paused

Screenshot 2023-07-12 at 17 43 22

@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 21:37 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 21:37 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 21:37 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 21:37 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 12, 2023 21:47 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 12, 2023 21:50 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 13, 2023 13:14 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 13, 2023 13:14 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 13, 2023 13:19 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 13, 2023 13:19 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 13, 2023 13:19 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 13, 2023 13:19 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Testnet July 13, 2023 13:28 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo temporarily deployed to Cloudflare-Preview July 13, 2023 13:29 — with GitHub Actions Inactive
@sebastianscatularo sebastianscatularo merged commit f8e14c2 into main Jul 13, 2023
9 of 10 checks passed
@sebastianscatularo sebastianscatularo deleted the feat/add-terra-classic-bridge-warning branch September 8, 2023 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@M-Picco M-Picco M-Picco approved these changes

@valentinoConti valentinoConti valentinoConti approved these changes

@raop155 raop155 Awaiting requested review from raop155

@kev1n-peters kev1n-peters Awaiting requested review from kev1n-peters

Assignees

@sebastianscatularo sebastianscatularo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sebastianscatularo @M-Picco @valentinoConti