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

update contract verification docs to be more precise #345

Closed
wants to merge 1 commit into from

Conversation

kaseLunt
Copy link
Contributor

@kaseLunt kaseLunt commented Mar 5, 2024

What changed? Why?

  • Updated the "Contract Verification" article in response to issue #342
  • Added direct links to sepolia.basescan.org throughout the article for clarity
  • Clarified language to ensure users understand they should use sepolia.basescan.org for contract verification and interaction on the Base Sepolia testnet
  • Improved sentence structure and overall flow while maintaining all necessary information

Notes to reviewers

How has it been tested?

  • Reviewed the updated article for clarity, accuracy, and completeness
  • Tested all links to ensure they lead to the correct pages on sepolia.basescan.org
  • Followed the updated instructions to verify and interact with a contract on sepolia.basescan.org to confirm the process works as described

Does this PR add a new token to the bridge?

  • [x ] No, this PR does not add a new token to the bridge
  • I've confirmed this token doesn't use a bridge override
  • I've confirmed this token is an OptimismMintableERC20

@wbnns
Copy link
Member

wbnns commented Mar 6, 2024

@kaseLunt

Thanks for taking the time to open this PR! We've added this to our backlog, pending to be reviewed.


You can interact with your deployed contract using Remix, the same as before, but it's also possible to interact with it through Etherscan. Paste your address in the search field to find it.
While you can interact with your deployed contract using Remix, you can also do so through [sepolia.basescan.org](https://sepolia.basescan.org). To find your contract, paste its address into the search field and click search.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change to reference-style links. See below for an example!

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

@briandoyle81CB briandoyle81CB left a comment

Choose a reason for hiding this comment

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

Thank you, this is a great PR! Requesting a couple minor changes

@wbnns
Copy link
Member

wbnns commented Apr 10, 2024

@kaseLunt Hi there! Since it's been some time since @briandoyle81CB left feedback without further changes, just wanted to follow-up to see if you were still interested in pursuing this PR.

hide_table_of_contents: false
---

Once your contract is deployed, you can verify it using a number of popular services. Doing so will let you users have confidence that your contract does what you claim, and will allow you to interact with it using a similar interface to what you used in Remix.
After deploying your contract, you can verify it using popular explorers like [basescan](https://sepolia.basescan.org). Verifying your contract allows users to trust that it does what you claim and enables you to interact with it using a user-friendly interface similar to Remix.
Copy link
Member

Choose a reason for hiding this comment

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

s/using popular explorers/using a popular explorer/

s/basescan/Basescan/

Copy link
Member

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

@kaseLunt

Please see the in-line comments with requested changes if you'd still like to pursue this improvement to the docs. 👍

@wbnns wbnns added change(s) requested One or more changes have been requested state: in progress and removed state: backlog labels May 23, 2024
@wbnns wbnns self-assigned this May 23, 2024
@wbnns
Copy link
Member

wbnns commented Jun 6, 2024

@kaseLunt

Hello, since it has been some time and we haven't heard back, I'm going to go ahead and close this PR for now. If you'd like to pursue these changes in the future please let us know and we can reopen. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change(s) requested One or more changes have been requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants