Skip to content

Conversation

@Viterbo
Copy link
Contributor

@Viterbo Viterbo commented Jan 22, 2025

Fixes #833

Description

This PR removes any reference to pTokens services and tries to avoid bringing ways for the user to send tokens to the wrong destinations.

image
image
image

@Viterbo Viterbo self-assigned this Jan 22, 2025
@Viterbo Viterbo linked an issue Jan 22, 2025 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for wallet-develop-mainnet ready!

Name Link
🔨 Latest commit 37aa02f
🔍 Latest deploy log https://app.netlify.com/sites/wallet-develop-mainnet/deploys/679160e4ac61a20008f7ff37
😎 Deploy Preview https://deploy-preview-834--wallet-develop-mainnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for wallet-staging ready!

Name Link
🔨 Latest commit 37aa02f
🔍 Latest deploy log https://app.netlify.com/sites/wallet-staging/deploys/679160e421e7470008deda7c
😎 Deploy Preview https://deploy-preview-834--wallet-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Viterbo Viterbo force-pushed the 833-remove-ptokens-from-the-telos-native-code branch from d33c78d to 0435420 Compare January 22, 2025 18:50
@pmjanus
Copy link
Contributor

pmjanus commented Jan 22, 2025

@Viterbo On the text for the small banner we can use: To transfer your TLOS tokens across blockchains, first send them to the Telos EVM network (click on >>> EVM), then connect to bridge.telos.net to complete your transfer.

If we can reduce the size of the box a little bit I think that would help, if not, that's ok.

image

Keep the hyperlink. Good job on that.

poplexity
poplexity previously approved these changes Jan 22, 2025
Copy link
Member

@poplexity poplexity left a comment

Choose a reason for hiding this comment

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

Looks good to me, pending @pmjanus suggestion 👍

@pmjanus pmjanus self-requested a review January 22, 2025 22:45
@pmjanus pmjanus merged commit 8218c2d into develop Jan 22, 2025
9 checks passed
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.

Remove pTokens from the Telos Native code

4 participants