-
Notifications
You must be signed in to change notification settings - Fork 5
Update Code Challenge #12
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
base: master
Are you sure you want to change the base?
Conversation
ddcf77c to
9b59148
Compare
src/app/containers/Countries/saga.ts
Outdated
| } | ||
| } catch (err) { | ||
| yield put(actions.fetchCountriesError(err.toString())); | ||
| } catch (err: any) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure what type this should be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found this from Mr. Dodds: https://kentcdodds.com/blog/get-a-catch-block-error-message-with-typescript
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love it
…nguageswitch, a, link, navbar, formlabel
| 8. Add an error view in case the data fetching has a problem. | ||
|
|
||
| This added container should function roughly like the existing `Countries` container. In the interview, we'll discuss this app, ways to improve it, architectural concepts, best practices and what ideas you think are important in making a strong, reliable and easy to use app. | ||
| 9. Style the page with tailwind. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will probably need to leave this off until we add in Tailwind
inlined mock dispatch
53c25a1 to
8c3afc6
Compare
Uh oh!
There was an error while loading. Please reload this page.