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

Add tips modal and text copies #31

Merged
merged 3 commits into from
Aug 30, 2023
Merged

Add tips modal and text copies #31

merged 3 commits into from
Aug 30, 2023

Conversation

Franpastoragusti
Copy link
Collaborator

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)

Copy link
Collaborator

@escottalexander escottalexander left a 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"}
Copy link
Collaborator

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."
Copy link
Collaborator

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."
Copy link
Collaborator

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."
Copy link
Collaborator

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."
Copy link
Collaborator

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."

@irfanbozkurt irfanbozkurt merged commit a395d7a into irfanbozkurt:develop Aug 30, 2023
escottalexander added a commit to escottalexander/flashbot-recovery-bundler that referenced this pull request Aug 27, 2024
* 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]>
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.

Add success social sharing Add copy texts Create tips modal
3 participants