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

add description for change_bid_public_key entry point #1479

Merged

Conversation

wojcik91
Copy link

What does this PR fix/introduce?

Adds a page describing the new change_bid_public_key entry point of system auction contract

Copy link
Contributor

@ACStone-MTS ACStone-MTS left a comment

Choose a reason for hiding this comment

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

Some minor edits, otherwise looks good!

@ipopescu
Copy link
Collaborator

ipopescu commented Aug 5, 2024

Great content, @wojcik91. Are you planning to finish this PR?

@wojcik91
Copy link
Author

wojcik91 commented Aug 6, 2024

@ipopescu I apologize for the delay, I must have missed some notifications. The stuff you pointed out is now fixed.

Copy link
Collaborator

@ipopescu ipopescu left a comment

Choose a reason for hiding this comment

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

@wojcik91, thank you for the updates. Could you please also change all deploys to transactions, including the casper-client command, which should use put-transaction? Please also test it.

Since deploy is deprecated, we are updating all 2.0 documentation to use transaction instead of deploy.

@wojcik91
Copy link
Author

wojcik91 commented Aug 7, 2024

@ipopescu I removed all references to deploys, but I still have one outstanding issue - the new put-txn command is in reality not a 1:1 translation of put-deploy.
I did actually use it to deploy a contract on devnet, but I did not figure out a way to correctly call installed contracts without using WASM code yet. Does some documentation for this command exist already?

@ipopescu
Copy link
Collaborator

ipopescu commented Aug 8, 2024

@ipopescu I removed all references to deploys, but I still have one outstanding issue - the new put-txn command is in reality not a 1:1 translation of put-deploy. I did actually use it to deploy a contract on devnet, but I did not figure out a way to correctly call installed contracts without using WASM code yet. Does some documentation for this command exist already?

Thanks, @wojcik91. This PR is ready for merging. We are still working on documentation for 2.0, including calling contracts. Let's merge this content and come back to the casper-client command once the docs for calling contracts are available.

@ipopescu ipopescu self-requested a review August 8, 2024 10:59
@ipopescu ipopescu merged commit d369c32 into casper-network:feat-2.0_docs Aug 9, 2024
8 checks passed
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.

3 participants