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

Remix Plugin: Make constructor arguments optional #1751

Open
kuzdogan opened this issue Nov 26, 2024 · 0 comments
Open

Remix Plugin: Make constructor arguments optional #1751

kuzdogan opened this issue Nov 26, 2024 · 0 comments

Comments

@kuzdogan
Copy link
Member

When watching the video below I noticed the constructor arguments popped up and later in the video it causes some confusion.

I think we should make this clear that this is not required. AFAIK this is only needed for Etherscan. And even for Etherscan this might not be required. Let's check if it's indeed possible to verify without these on Etherscan. If so make it really clear that this field is optional. If it's required for Etherscan, let's only show this when Etherscan is enabled and let the user know this is needed for Etherscan. We know Sourcify does not require this (although we appreciate if provided), we should also check Blockscout and Routescan for that.

image
@marcocastignoli marcocastignoli moved this from Triage to Backlog in Sourcify Public Nov 26, 2024
@marcocastignoli marcocastignoli moved this from Backlog to Sprint - Candidates in Sourcify Public Nov 26, 2024
@marcocastignoli marcocastignoli moved this from Sprint - Candidates to Backlog in Sourcify Public Nov 26, 2024
@marcocastignoli marcocastignoli moved this from Backlog to Sprint - Candidates in Sourcify Public Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Sprint - Candidates
Development

No branches or pull requests

1 participant