-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add tips modal and text copies #31
Add tips modal and text copies #31
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some suggestions on wording changes. We can discuss if you don't like them. Nice work. this is looking great!
@@ -72,7 +85,7 @@ export const RecoveryProcess = ({ | |||
return ( | |||
<CustomPortal | |||
title={"Switching Network"} | |||
description={"Switch to personal Flashbot RPC network to prepare the transacion you will pay"} | |||
description={"Allow us to create and switch to your personal RPC network to prepare the transaction bundle"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The user doesn't need to know that we are creating the personal RPC and they don't need to know about bundles. Maybe we could simplify this while also giving them some safety assurance by linking to flashbots.
Proposed wording: "Now you are being switched to a new Flashbots(link to https://www.flashbots.net/) RPC. This will enable you to send transactions that are undetected by the hacker until after your assets have been recovered. Please approve the network change in your wallet after you read over the RPC details and feel safe moving forward."
It's a little wordy but I feel like it explains just enough to help the user know what is happening and also buys some clout by linking to a reputable site. Users will probably be on high alert after just being hacked so we want to ease their fears as much as possible.
@@ -82,7 +95,9 @@ export const RecoveryProcess = ({ | |||
return ( | |||
<CustomPortal | |||
title={"Increase the gas"} | |||
description={"Switch to personal Flashbot RPC network to prepare the transacion you will pay"} | |||
description={ | |||
"We recommend set a hight gas to ensure a successful asset recovery. Your increased gas allocation will facilitate the seamless completion of the transaction." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"We recommend to use a custom gas price to ensure successful asset recovery."
description={ | ||
"Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nunc vulputate libero et velit interdum, ac aliquet odio mattis. Class aptent taciti." | ||
"To proceed with asset recovery, switch to the compromised address in your wallet. This step is essential to verify ownership and continue with the recovery process." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"To proceed with asset recovery, switch to the compromised wallet. This step is essential to verify ownership and continue with the recovery process."
description={ | ||
"Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nunc vulputate libero et velit interdum, ac aliquet odio matt" | ||
"Your asset recovery is in progress. To ensure a successful retrieval and accurate transaction completion stay on the page." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would remove "and accurate transaction completion".
@@ -139,15 +154,57 @@ export const RecoveryProcess = ({ | |||
<CustomPortal | |||
title={"Your assets have been recovered!"} | |||
description={ | |||
"If we have helped you please, share the tool and if you want let us a tip to continue contributing" | |||
"Check your safe wallet for your retrieved assets. Share your journey and consider to support us with a tip to continue serving the crypto community." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Check your safe wallet for your retrieved assets. If you were helped by this tool, please consider sharing this on social media and if you are feeling generous, consider giving us a small tip so we can continue providing helpful tools to the crypto community."
* add impersonator * add abininja flow * add custom-abininja type for future styling * remove add btn and close modal as soon as we get latestTransaction * style tweaks and display custom-abininja properly --------- Co-authored-by: escottalexander <[email protected]>
This PR improve the curren texts and explanations and also adds the tips step after success and the links to share in twitter and telegram (we can add more)