Pob Ch

change a regex and contents on Redirect on login chapter

3

- We always call `queryString()` with `"redirect"` as an argument, so I think it's not necessary to use regex to escape special characters on `name` variable. (and you forgot to...

set initial state values to be matched with prior chapters' standard

1

On previous chapters, we always set `isLoading` initial value to `false`. So, I think we should do the same here.

useQuery `pollInterval` stop working in React v18 StrictMode

6

**Intended outcome:** `pollInterval` should work either with React v18 StrictMode or without StrictMode **Actual outcome:** `pollInterval` stop working when using it with React v18 StrictMode **How to reproduce the issue:**...

Documenting about why we shouldn't add a shared service into multiple modules without `imports`

2

## I'm submitting a... - [ ] Regression - [ ] Bug report - [ ] Feature request - [x] Documentation issue or request (new chapter/page) - [ ] Support...

Mutating state

1

I think you are mutating the state at the following lines: https://github.com/dabit3/full-stack-serverless-code/blob/44c4eb269757d412f8fb27a9cee4a7bb79bf768b/graphql/src/App.js#L83-L84 Since each `note` is an object, setting `notes[index].completed` property is mutating the `note` object. Instead, we should create...

- **File(s) Modified**: _0104-maximum-depth-of-binary-tree.js_ - **Language(s) Used**: _javascript_ ### Description Correct the space complexity.

- **File(s) Modified**: _0235-lowest-common-ancestor-of-a-binary-search-tree.js_ - **Language(s) Used**: _javascript_ ### Description The time complexity of the current solution should be O(H), **not** O(N) when N is the total number of node...

Add "How to uninstall elm" section

1

There is only "How to install elm" section: https://guide.elm-lang.org/install/elm.html The missing part is how to uninstall it for each OS (Mac, Linux, Windows).