Is this the perfect React app? - Code Review

You are not supposed to use objects or arrays in the dependency array: ```console.log({} === {}) // false```.
The showcased use of useMemo and useCallback is pointless as it would still rerun on every rerender.

I am not a fan of export default

Im inspired to create my own small social media app too. Im a mid level fullstack developer who wants to be senior soon

Storing an access token in localstorage is not a good practice though.

All the idiots here knitpicking on the code, for what its worth , its pretty good and understandable 😁, its good enough code thats worth to ship 🚢

Where is the TypeScript?

I spotted numerous flaws. Where can I add my CR comments?

Hey, so I have a question. Which is a better practice : storing authToken in local storage or cookie and why?

Is the useSyncExternalStore important in react hooks ?

why it passes the userData as props when it could have got that data inside the component itself?

I really enjoyed the video!
As a junior web developer, I appreciate your advice to learn React despite the negative feedback that some users have given it. I'm a little confused as to why people talk negatively about React, since I'm not very familiar with the "dev culture". It would be great if you could provide some guidance to help clear up any confusion.
Thank you!

Putting your access token in the local storage makes your site vulnerable to csrf and xss.
The best practice will be to use cookies for authentication.

tbh, its not that clean or good - relative imports are all over the place - a complex codebase can undergo a lot of refactors - using aliases for paths is always important in such cases

Does anyone know a perfect nextjs 13 app like this? I would like to learn from it.

❤👌

How do I submit a app for review?

Would it make sense to move the useEffects in app container into seperate hooks that way you can clearly see what each one does and they aren’t all in the app container?

There is no test case. Also we can use absolute path for import which is missing in the codebase

Wait, what. Can anyone enlighten me why the accessToken is saved on localStorage????

Thank you so much for this video and the owner of repo, it helps a lot! Btw, can I know what is the extension you are using that make the top wrapper name/function pinned at the top? Thanks!!

There is no such thing as perfect, there is no silver bullet.

Honestly this is kind of bad example because it contains so many problems. At first there are a lot of useEffects going on which do some request and then setting state (setLoadings...) without dealing with unmounting... Additionally looking at the redux part, it is so much boilerplate without adding any value at all, most of the actions/reducers are just fetching data and updating the store. Secondly storing your (access/refesh) tokens in localstorage is just a big security concern. And there are more mistakes which are mostly made by non senior devs...

Great content, but I can not say that not using a framework is react best practices.

Not using useMemo does not invalidate memo..
The only thing you are optimising with useMemo there is just mapping over an array

can you do a video on your vscode config? 🙏

what vscode theme is this?

I'll love to rebuild this using and I'll want you to review it

Sir please make a video of MERN full stack app of e-commerce site❤..From basic to advance

To be honest there is so much wrong with this app, cannot believe that dumping server state in redux is still a thing, jeeez...

lost 20 iq watching the thumbnail

Hey Bro, i'm building one application in reactjs, how i can reach you for reviewing that app

Hey, what is your vscode theme?

This is like 2018 - 2019 grade react 💩... U literally will be fired if u write this in 2023

It's nice and follows OK practices. TypeScript is a must and should be a GO TO for every new application.

I am working on similar structure everything is so well organized and we are using ts as well.

how can i achieve this kind of clean code and architecture compared to my current shit code ? how do i imporove myself

why is he storing the auth token in local storage !! is this safe/okay ?

the project doesn't use ESLint at all, for me this is very bad, the "let vs const" situation would be easily found and fixed with the right eslint rules

Damn once I saw this code structure I have a lot to learn

I think putting store creation inside an effect doesn't make any sense, there's no need for that, just create the store outside of that component (import it or above the file), then inject it into the provider, apart from that it seems a good one

Bad practice to store auth info in localstorage.

Please make video over protected user role base routing in react js

Nice🎉

It's a fairly big and complex codebase, if he can code this clean and maintainable without typescript, it's even more impressive 🔥

Thank you so much the owner of repo and Cosden for this ! Altho i was disappointed to not see typescript being used, but the code is clean and easy to read. got inspired and learned a lot.

keep up 💪💪 bro.

I don the front end of this website 😁