Fix: New disclaimer: Do not send to exchanges on certain networks (AH) #8149

[AshutoshSingh00001]

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

👇 __ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Transfer: New disclaimer: Do not send to exchanges on certain networks (AH) #8149
• ## Did your issue had any of the "$" label on it?
• My DOT address: Payout

[kodabot]

SUCCESS @AshutoshSingh00001 PR for issue #8149 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime!

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	0c3ed09
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/6561b7c6ed40a8000873b8da
😎 Deploy Preview	https://deploy-preview-8180--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[reviewpad]

AI-Generated Summary: This pull request introduces several changes to the Transfer.vue and en.json files. The changes in Transfer.vue include the addition of a new info-tooltip and warning exchange toggle, along with corresponding logic for managing the tooltip and switch states. The "continue" button has also been updated to be disabled not only when a "disabled" condition is met, but also when the warning exchange toggle is turned off. The reference variable 'warningexchange' has been renamed to 'warningExchange' for better readability. In addition, a new warning message, "Do not send tokens to centralized exchanges", has been added to the en.json file for localization purposes.

[AshutoshSingh00001]

@JustLuuuu @prury please review

[JustLuuuu]

@AshutoshSingh00001

I wanted to drop my bit here.

• It's not very clear why people should not send the funds to exchange.
• We are not offering any solution to them (what should they do if they want to send funds to exchange?)
• The hovering info point has the same text as the main text.
• I would put a different text under the info circle: Something like if you are sending your funds from Assethub to exchange, your funds may be permanently lost; teleport them before sending. Or something in this style

[AshutoshSingh00001]

@AshutoshSingh00001

I wanted to drop my bit here.

• It's not very clear why people should not send the funds to exchange.
• We are not offering any solution to them (what should they do if they want to send funds to exchange?)
• The hovering info point has the same text as the main text.
• I would put a different text under the info circle: Something like if you are sending your funds from Assethub to exchange, your funds may be permanently lost; teleport them before sending. Or something in this style

thanks for the reply @JustLuuuu
i'lll be updating my pr according to your needs

[JustLuuuu]

AAA text ran away from the info box 😂

[prury]

mobile needs work as well:

[exezbcz]

Uf, I'm not sure about this.

Sorry, but I did not drop any instructions, now it's just a random toggle button that does not make sense.

#8149 (comment)

[codeclimate]

Code Climate has analyzed commit 97cf8b8 and detected 0 issues on this pull request.

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information