Activity
Mon
Wed
Fri
Sun
Sep
Oct
Nov
Dec
Jan
Feb
Mar
Apr
May
Jun
Jul
What is this?
Less
More

Memberships

Dev Mastery Roadmap

3.3k members • Free

Selftaught Developer Incubator

377 members • $49/m

Developer Pro

22 members • $250/m

711 contributions to Developer Pro
React Challenge 2
https://codesandbox.io/p/sandbox/react-challenge-2-with-tests-forked-8d57pc
3 likes • 30d
looks pretty good, but you can improve it a bit: - trim() the input before submit - early return if empty value is submitted - handleToggle mutates state directly (I'll let you figure it out on your own)
CRM for leads projects
https://github.com/mhayakawa1/crm-for-leads
1 like • Jun 23
looks pretty good, but you have some big oopsies on the backend api side: middleware does pretty much nothing (it checks if a Authorization header exists, but you don't check the token so every endpoint could be used with a 'haha-this-is-a-token' Authorization header), for some reason you took a perfect JWT created by supabase, and use it to sign another JWT that you created, you store the token in localStorage (though you have the cookies infra set up perfectly, you should stay with it imo), you throw errors on the server and return 400 (I saw this in a try-catch block somewhere where you correctly threw an error with 403 after doing some checks on the user, but then in the catch block you assigned the same status code, which is not correct, you should make sure the client side errors are handled and throw 500 with minimal info in the console for the others), you check plain text passwords (never ever do that, compare their hashes), validate everything that comes from the frontend. I would do a deep review of the backend if I were you and try to improve it a bit.
PR-converter page
https://github.com/SherpaMilan/cryptoApp/pull/26 @Cristian Florea here i think i did lots of work. try to make as best as possible. This converter has a way to search coins if it is not in the local list. we can mark favourite , recent coins goes to recent column, it supports default currency change, please have a look, thank you
PR-converter page
1 like • Jun 13
This looks sick bro, well done ✅
Console errors
Do we keep them in our projects or not? What's the best practice?
2 likes • Jun 3
If you catch the errors and handle them, the error logs they are not the end of the world, but console logs should be for dev env only
Coding Update
Recently Completed - Assign users to leads - Add user id relation in database - Only allow users to view their own data Current tasks - Create kanban board - Convert lead statuses with board
0 likes • May 29
Fyi user_id is usually the foreign key, the relation naming convention is just user/users, depending on the type. Have you deployed this app?
1-10 of 711
Adelin Rosca
6
106 points to level up
@adelin-rosca-5395
Trying to break free from the shackles that prevent me from living my life, and becoming a web developer is the path to my liberty

Active 12h ago
Joined May 16, 2024
Romania