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 demo contract #353

Merged
merged 11 commits into from
Sep 15, 2020
Merged

Update demo contract #353

merged 11 commits into from
Sep 15, 2020

Conversation

chenzhitong
Copy link
Member

close #351

@chenzhitong chenzhitong changed the title Issue 351 Update demo contract Sep 11, 2020
Tommo-L
Tommo-L previously approved these changes Sep 11, 2020
templates/Template.CSharp/Contract1.cs Outdated Show resolved Hide resolved
// this method will be triggered as a VerificationTrigger to verify that the signature is correct.
// For example, this method needs to be called when withdrawing token from the contract.
public static bool Verify() => IsOwner();

public static bool Main()
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public static bool Main()
public static bool MyMethod()

Copy link
Member

Choose a reason for hiding this comment

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

And we can add some comments to explain that this should be replaced with your own methods.

Copy link
Member

Choose a reason for hiding this comment

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

Add comments.

templates/Template.CSharp/Contract1.cs Outdated Show resolved Hide resolved
templates/Template.CSharp/Contract1.cs Outdated Show resolved Hide resolved
shargon
shargon previously approved these changes Sep 11, 2020
@shargon shargon merged commit 2ebb098 into neo-project:master Sep 15, 2020
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.

Update demo contract
4 participants