-
Notifications
You must be signed in to change notification settings - Fork 193
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
add statemanagement data and illo #461
Conversation
This pull request is being automatically deployed with Vercel (learn more). egghead-io-nextjs – ./🔍 Inspect: https://vercel.com/eggheadio/egghead-io-nextjs/kk198kw7r egghead-next-storybook – ./🔍 Inspect: https://vercel.com/eggheadio/egghead-next-storybook/atpy3ddc3 |
@@ -395,12 +397,31 @@ There are a number of tried and true methods that will allow you to learn more e | |||
path: | |||
'/lessons/react-using-recoil-to-manage-orthogonal-state-in-react-apps-with-david-mccabe?pl=react-state-management-2020-6bec', | |||
}, | |||
{ |
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.
missing a couple of interviews on card for homepage
<CardVerticalWithStack data={devEssentials} /> | ||
|
||
<CardVerticalWithStack data={stateManagement} /> | ||
<Card resource={tailwind} className="text-center"> |
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.
moved the 'aside' cards around
<ol className="text-left"> | ||
{tailwind.resources.map((resource: any, index: any) => { | ||
{stateManagement.resources.map((resource: any, index: 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 using CardVerticalWithStack so the image is displayed
} | ||
alt="illustration for state management in react" | ||
width={150} | ||
height={150} | ||
width={200} |
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.
This illo looked a bit better to me when it was a little bigger
guid: '6732', | ||
slug: 'react-state-management-in-2021-6732', | ||
illustrator: { | ||
name: 'Maxime Bourgeois', |
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.
Maxime is the illustrator!
freshness: { | ||
status: `fresh`, | ||
title: `This is a Fresh Resource`, | ||
text: `This series digs into the deeper patterns of development with leading experts in the field. It's worth your time.`, |
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.
review
src/data/courseDependenciesData.js
Outdated
}, | ||
essentialQuestions: [ | ||
'How do you know when you need to use a state management library?', | ||
'Should all state changes be the result of an event and an action?', |
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.
pulled from summaries
pairWithResources: [ | ||
{ | ||
id: 432472, | ||
title: 'Introduction to State Machines Using XState', |
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.
thought suggesting state management courses here made sense
changes made
Screen shots