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 and remove links to old examples #260

Merged
merged 3 commits into from
Feb 9, 2024

Conversation

Ben-Rey
Copy link
Contributor

@Ben-Rey Ben-Rey commented Jan 10, 2024

No description provided.

@Ben-Rey Ben-Rey requested a review from sydhds January 10, 2024 15:02
@Ben-Rey Ben-Rey linked an issue Jan 10, 2024 that may be closed by this pull request
@Ben-Rey Ben-Rey requested a review from Leo-Besancon January 10, 2024 15:29
Copy link
Contributor

@Leo-Besancon Leo-Besancon left a comment

Choose a reason for hiding this comment

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

This link still leads to 404:
https://github.com/massalabs/web3-utils/blob/main/src/arguments.ts#L42
("You can learn more about Args via the source code")

Also: "A complete example of how to use the Wallet API is also available."

@Ben-Rey Ben-Rey force-pushed the 259-link-to-example-in-massa-web3-is-not-valid branch from fe94fbd to 8ac1471 Compare January 10, 2024 18:11
@Ben-Rey Ben-Rey force-pushed the 259-link-to-example-in-massa-web3-is-not-valid branch from 8ac1471 to d61d377 Compare January 11, 2024 08:17
@Ben-Rey Ben-Rey requested a review from Leo-Besancon February 8, 2024 09:12
Copy link
Contributor

@Leo-Besancon Leo-Besancon left a comment

Choose a reason for hiding this comment

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

The only remaining thing for me is that the Mainnet tab here is broken, but we could just do it in another PR if you want to merge this one ;)
https://docs.massa.net/docs/build/massa-web3#without-wallet-provider

@Ben-Rey
Copy link
Contributor Author

Ben-Rey commented Feb 9, 2024

The only remaining thing for me is that the Mainnet tab here is broken, but we could just do it in another PR if you want to merge this one ;) https://docs.massa.net/docs/build/massa-web3#without-wallet-provider

Fixed thanks!

@Ben-Rey Ben-Rey merged commit 2b10430 into main Feb 9, 2024
4 checks passed
@Ben-Rey Ben-Rey deleted the 259-link-to-example-in-massa-web3-is-not-valid branch February 9, 2024 12:29
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.

Link to example in massa-web3 is not valid
2 participants