Skip to content

Cynthia Coronado - Custom Hook Project #70

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.

Sign up for GitHub

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

Open
cynthia-coronado wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Cynthia Coronado - Custom Hook Project #70

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
740be3b
Step one completed
cynthia-coronado Jul 13, 2021
4f7eb40
Step two completed
cynthia-coronado Jul 13, 2021
2bff51a
Made some changes, still not working
cynthia-coronado Jul 13, 2021
b4dd600
Finished! I had to move hooks to the src file, duh
cynthia-coronado Jul 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Follow these steps for completing your project.
- [ ] Submit a Pull-Request to merge <firstName-lastName> Branch into main (student's Repository). **Please don't merge your own pull request**
- [ ] From the home page of your repo, make sure you have your branch selected
- [ ] Copy the URL and paste it into Canvas

## Minimum Viable Product

- [ ] Build a custom hook that let's you save data to localStorage
Expand Down Expand Up @@ -84,7 +84,7 @@ export const useLocalStorage = (key, initialValue) => {
- `setValue` should look something like this:

```js
const setValue = value => {
const setValue = (value) => {
// Save state
setStoredValue(value);
// Save to local storage
Expand Down
Loading