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

Rename interceptor in manifest and add a better description #1048

Merged
merged 4 commits into from
Jun 21, 2024

Conversation

KillariDev
Copy link
Contributor

No description provided.

@KillariDev KillariDev added the enhancement New feature or request label Jun 19, 2024
@KillariDev KillariDev requested a review from jubalm June 19, 2024 12:17
Copy link
Collaborator

@jubalm jubalm left a comment

Choose a reason for hiding this comment

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

I feel that the "benefit" aspect of the tool would be important to the description being "risk-free"

Maybe @MicahZoltu would like to leave his thoughts as well before we merge it

"name": "Interceptor",
"description": "Interceptor",
"name": "The Interceptor",
"description": "Gain a clearer understanding of your Ethereum transactions with The Interceptor browser extension",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Offering a description suggestion

Suggested change
"description": "Gain a clearer understanding of your Ethereum transactions with The Interceptor browser extension",
"description": "Know exactly what is happening with your Ethereum transactions without putting anything at risk",

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Gain a clearer understanding of your Ethereum transactions with The Interceptor browser extension",
"description": "Gain a clearer understanding of your Ethereum transactions before any of your assets are at risk. Simulate a series of transactions to see what will happen before signing anything. Get warned of common scams and mistakes before you become a victim of them.",

@DarkFlorist DarkFlorist deleted a comment from jubalm Jun 21, 2024
@KillariDev KillariDev merged commit 9dd1d31 into main Jun 21, 2024
1 check passed
@KillariDev KillariDev deleted the manifest-update branch June 21, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants