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

Eventonica Updates #1629

Merged
merged 1 commit into from Sep 13, 2021
Merged

Eventonica Updates #1629

merged 1 commit into from Sep 13, 2021

Conversation

dpenny
Copy link
Collaborator

@dpenny dpenny commented Sep 12, 2021

updating some of the tutorials after seeing people go through Eventonica last week:

  • more detailed walkthrough for the getUsers endpoint - add information about the req.body, using postman, and naming

    • I want to encourage people to use Postman, since it can make debugging easier, and is a good skill for API development
  • an example of useReducer where multiple fields are updated by one action

    • I feel like complex updating of state is one of the main benefits of useReducer -- the tutorial currently only shows one field at a time, so add another section about this
  • make mock event IDs be strings, not int

    • saw some issues where people were confused because the input sends back an ID as a string, but the event IDs are stored as numbers. I feel like this was an unnecessary hurdle, and making them strings will make things flow smoother

@dpenny dpenny requested a review from alodahl September 12, 2021 21:15
@alodahl alodahl merged commit 135a2da into Techtonica:main Sep 13, 2021
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.

None yet

2 participants