You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Absolutely love the concept!! Your codebase looks really neat, and I was particularly a fan of how far you streamlined processes. For example:
functionusers(req,res,isWizard){console.log(!isWizard)consttitle='PactPortal';constusers=getUserList(isWizard ? 0 : 1);// defining your query through this simple boolean is super readable and re-usableconstcontent=/*html*/` <div class="banner"> <div> <a href="/user"><button class="button">Profile</button></a> <a href="/log-out"><button class="button">Log Out</button></a> </div>${users.map(user=>createPost(user))} // exactly the same line of code for either result! I love it <3 </div>`returnlayout({ title, content });}
Be careful with cleanups though. This is a tiny example, but ghost code like this creeps up and leads to overload and confusion:
If something is commented out by the time you're ready to push just remove it.
Nice simple fix here, where you were struggling for a bit in the day:
functiondeleteUser(id){console.log(`deleting user: ${id}`);// console.logs are dirty and I hate them thoughremoveSessionByUserId(id);returndelete_user.run({ id });}
The text was updated successfully, but these errors were encountered:
Absolutely love the concept!! Your codebase looks really neat, and I was particularly a fan of how far you streamlined processes. For example:
Be careful with cleanups though. This is a tiny example, but ghost code like this creeps up and leads to overload and confusion:
If something is commented out by the time you're ready to push just remove it.
Nice simple fix here, where you were struggling for a bit in the day:
The text was updated successfully, but these errors were encountered: