Skip to content
This repository has been archived by the owner on Jul 16, 2020. It is now read-only.

update exercises for clarity #21

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

zacjones93
Copy link
Contributor

We had some great feedback on where the exercises were unclear. Went through the exercises with the feedback in mind to build up each exercise better.

@zacjones93
Copy link
Contributor Author

@all-contributors please add @doingandlearning for userTesting

@allcontributors
Copy link
Contributor

@zacjones93

I've put up a pull request to add @doingandlearning! 🎉

@@ -1,4 +1,4 @@
id,first_name,last_name,email,gender,age,language
user_handle,first_name,last_name,email,gender,age,language
1,Ophelia,Claffey,[email protected],Female,40,Macedonian

Choose a reason for hiding this comment

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

would the values change here?

Choose a reason for hiding this comment

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

it changed above to uuid but remained an int

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants