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

Ccip/1.5.1 remix #2205

Merged
merged 63 commits into from
Dec 13, 2024
Merged

Ccip/1.5.1 remix #2205

merged 63 commits into from
Dec 13, 2024

Conversation

aelmanaa
Copy link
Collaborator

@aelmanaa aelmanaa commented Dec 5, 2024

No description provided.

Copy link
Collaborator

@thedriftofwords thedriftofwords left a comment

Choose a reason for hiding this comment

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

Non-blocking feedback:

Prerequisites

  • Web browser setup: Add link or reminder to check supported networks page
  • Native gas tokens: Add link to billing page (for users who need guidance on how much gas tokens are enough

Blockchain setup

  • UI nit: The "<>" between the dropdown fields is clickable which might lead users to expect they can swap the values of the fields by clicking that

Copy link
Collaborator

@thedriftofwords thedriftofwords left a comment

Choose a reason for hiding this comment

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

Excellent 🚀

aelmanaa and others added 6 commits December 13, 2024 18:39
Co-authored-by: Crystal Gomes <thedriftofwords@users.noreply.github.com>
…a-remix.mdx

Co-authored-by: Crystal Gomes <thedriftofwords@users.noreply.github.com>
Co-authored-by: Crystal Gomes <thedriftofwords@users.noreply.github.com>
Co-authored-by: Crystal Gomes <thedriftofwords@users.noreply.github.com>
Co-authored-by: Crystal Gomes <thedriftofwords@users.noreply.github.com>
@aelmanaa aelmanaa merged commit 881400d into main Dec 13, 2024
10 checks passed
@aelmanaa aelmanaa deleted the ccip/1.5.1-remix branch December 13, 2024 18:24
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.

3 participants