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

Verification plugin: Add new icon and feature plugin #5601

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

manuelwedler
Copy link
Contributor

We wanted to give the plugin a prettier icon.
Besides this, it would be nice if we could feature the plugin on the home view of Remix, since it is still quite new.

Copy link

netlify bot commented Jan 3, 2025

👷 Deploy request for remixproject pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1875e07

@@ -26,6 +26,7 @@
"home.solhintPluginDesc": "Solhint is an open source project for linting Solidity code.",
"home.sourcifyPluginDesc": "Solidity contract and metadata verification service.",
"home.unitTestPluginDesc": "Write and run unit tests for your contracts in Solidity.",
"home.contractVerificationDesc": "Verify contract code on Sourcify, Etherscan, Blockscout and Routescan at the same time.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

This test needs to be slightly shorter. Try this"
Verify contracts on multiple services at the same time.

})}
maintainedBy="Remix"
callback={() => startContractVerification()}
/>
<PluginButton
Copy link
Collaborator

Choose a reason for hiding this comment

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

The position of Solidity Analyzers should be swapped with the LearnEth promo. So that LearnEth appears 2nd to the left and Solidity Analyzers should be 3rd from left.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants