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

Satellite UI: zkSync updates #6873

Closed
mobyvb opened this issue Mar 20, 2024 · 6 comments
Closed

Satellite UI: zkSync updates #6873

mobyvb opened this issue Mar 20, 2024 · 6 comments
Assignees

Comments

@mobyvb
Copy link
Member

mobyvb commented Mar 20, 2024

UI changes:

  • We need to address the banner that claims we don't support zkSync payments
  • We need to make sure the "view" button on payment history tab routes to the correct block explorer.

AC:

  • correct link for zkSync transactions on token transactions list
  • remove text in the UI that claims that we only support L1 transactions - TODO what should the new text be? Do we need to warn against sending certain types of L2 transactions?
@mobyvb mobyvb added Needs Grooming Needs Estimation Issue still needs story pointing labels Mar 20, 2024
@ferristocrat
Copy link

What if we updated the warning to something like:

Send only STORJ tokens via Layer 1 or zkSync transaction to this address.
Sending anything else than STORJ token will result in the loss of your deposit.

@heunland
Copy link

Please specifically state in the warning message that we only accept ethereum based L1 or zksync Era STORJ deposits (unless we actually will also allow zkSync lite) and explicitly state that we do NOT accept STORJ deposits via other blockchains such as BNB, polygon etc since we did already have people try that.

@mobyvb
Copy link
Member Author

mobyvb commented Mar 25, 2024

@heunland @ferristocrat how does this look?

Send only STORJ tokens via Layer 1 or zkSync Era transaction to this address.
Sending anything else other than STORJ token, or using any Layer 2 other than zkSync, such as BNB, Polygon, etc... will result in the loss of your deposit.

Please let us know if this looks good. If it does not look good, please provide exact text that you would like to use instead.

@storj-gerrit
Copy link

storj-gerrit bot commented Mar 25, 2024

Change web/satellite: update STORJ deposit messaging mentions this issue.

@NiaStorj NiaStorj removed Needs Estimation Issue still needs story pointing Needs Grooming labels Mar 25, 2024
storjBuildBot pushed a commit that referenced this issue Mar 25, 2024
This change updates the STORJ deposit messaging to add support for
zkSync.

Issue: #6873

Change-Id: Ic3027b23a6c1a47f2289ad0efdf7e8ce6e2c18c5
@storj-gerrit
Copy link

storj-gerrit bot commented Mar 26, 2024

Change satellite/console: use correct link for zkSync mentions this issue.

storjBuildBot pushed a commit that referenced this issue Mar 28, 2024
This change uses the right block explorer link for zkSync transactions.

Issue: #6873

Change-Id: I49165e2b0becf45bc64ee7e6d04913e3a23f344f
andriikotko pushed a commit that referenced this issue Apr 2, 2024
This change uses the right block explorer link for zkSync transactions.

Issue: #6873

Change-Id: I49165e2b0becf45bc64ee7e6d04913e3a23f344f
andriikotko pushed a commit that referenced this issue Apr 2, 2024
This change uses the right block explorer link for zkSync transactions.

Issue: #6873

Change-Id: I49165e2b0becf45bc64ee7e6d04913e3a23f344f
@wilfred-asomanii
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants