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

Streamline counter, wCCD and voting smart contract tutorials #1043

Merged
merged 13 commits into from
Jul 30, 2024

Conversation

Victor-N-Suadicani
Copy link
Contributor

Purpose

Streamline and update the tutorials to make them simpler and easier to follow.

Changes

  • Rewrote counter tutorial to improve the wording and simplify the code.
    • Many subpar VSCode screenshots that felt unnecessary were removed.
    • The code was simplified to use a single update function rather than separate increment and decrement functions. A single error variant OverflowError is used instead of separate errors for the two previous functions.
    • Added instructions for how to use Concordium's testnet node to go through the tutorial, which makes it much easier to follow.
  • Slightly edited the wCCD tutorial, including merging the last part with the previous part since it is so short.
  • Some cleaning up for the voting tutorial, including making it adhere to the new simplified version, see Simplify voting example and default template concordium-rust-smart-contracts#440.
  • Removed some redundant/repeated sections in the piggybank tutorial and elsewhere.

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

@Victor-N-Suadicani Victor-N-Suadicani requested a review from DOBEN July 24, 2024 10:16
@DOBEN DOBEN added the preview Trigger a documentation preview for a PR. label Jul 26, 2024
Copy link

github-actions bot commented Jul 26, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-07-30 07:23 UTC

Victor Nordam Suadicani added 2 commits July 26, 2024 14:53
@Victor-N-Suadicani Victor-N-Suadicani requested a review from DOBEN July 26, 2024 12:55
@Victor-N-Suadicani Victor-N-Suadicani added preview Trigger a documentation preview for a PR. and removed preview Trigger a documentation preview for a PR. labels Jul 29, 2024
@Victor-N-Suadicani Victor-N-Suadicani merged commit 7798dc6 into main Jul 30, 2024
2 checks passed
@Victor-N-Suadicani Victor-N-Suadicani deleted the streamline_tutorials branch July 30, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Trigger a documentation preview for a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants