Live refactoring a subscribers Next.js application, plus giving my opinions

[@elmalleable]

Pretty nice. Just a tip for the review videos. I think you should use ppls names or pseudonyms if you ask them and they say no. It makes the video more relatable than always saying *the person* or he or him. It creates an invisible wall. But having a name generates more interest. And regularly point out good things you see or something new you learned from their code and maybe you can score them against yourself. That would be real cool and in boost the interest level of your videos. Nice catching yourself about saying crap code and correcting it. I'd work with you any day

[@WebDevCody]

That’s a great idea actually, I’ll start asking these people if I can use their names or some type of handle to make it more personal. Love it! Yeah I didn’t mean to say crap, but I think editing it out might have left the video choppy at a strange sentence. I try to keep it positive, but sometimes this code is overwhelming lol

[@yaseru]

I absolutely love this format, keep them coming!

[@stanimirdimitrov4913]

Amazing! I would have to take 2 days off work to watch all of your videos that I have missed 😆

[@WebDevCody]

Tap into that vacation time brother

[@stanimirdimitrov4913]

@@WebDevCody Really good stuff, Cody. Thank you!

[@ragnarlothbrok3228]

I love your content it’s so helpful

[@SeibertSwirl]

Good job babe! You’re doing so good!!

[@marcusabukari697]

Keep up the good work. Yeah the companies could definitely just be a computed value. No need to update the state.

[@NoFailer]

createStyle() is being called for every single company card component that is being rendered. I think it is much more efficient to create the style once and then passing it on to the cards.

[@WebDevCody]

Good catch, I’m not good at mantine or style components, so I wasn’t too sure what that was doing.

[@LikeFunnyMovies]

I think the value is cached by emotion anyways, so this shouldnt make a big difference

[@markopolo2224]

i love these videos so so much because they are so helpful !

[@FelipeRodrigues-zh3gx]

Thanks for the video!

[@sachinahya7502]

Barrel files are what they're called I think. The only time they're useful imo is when you're creating an npm package and you're declaring everything you're exposing publicly. Otherwise they're just one extra thing to maintain. They also look worse in ESM code when you have to add '/index.js' to the end of the path.

[@ThanHtutZaw3]

why my profilers dev tools didn't work. It says need react 18 but I am

[@BjornBrasse]

Why fetching all data clientside? Feels like missing the purpose of using NextJs

[@WebDevCody]

There are still benefits because next will statically render the individual page and tree shake only the dependencies it needs I believe, So the initial page will load will be much smaller. Also, there is a trade off between SSR and CSR. When you access any SSR page your browser will sit there just loading, so if it takes 2 seconds for your database to send back data, users get no feedback. At least in CSR you can actually show some type of initial info and show a spinner while more data is fetched from the api. It’s all about use cases and trade offs.

[@rohitkf8474]

His code looks way tooo overkill for something so simple. Correct me if I'm wrong

[@WebDevCody]

Yeah, I could see that. I think removing the callbacks and memos made it a bit simplier

[@elmalleable]

Maybe he wanted to play with lots of features and just have a feel. Very good strategy. Is it over kill? He's computer didn't die so....

[@vekaev]

This architecture gives easy scaling))

[@rohitkf8474]

@@elmalleable 🤣🤣🤣🤣