-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added upvotes on posts #160
base: main
Are you sure you want to change the base?
Conversation
* Bugfix/72/delete periods & small fixes (#151) * small design fixes * added cascade delete to application period * Bugfix/71/application form fix (#152) * application form fix * user data fix in application form * Feature/88/delete profile picture (#153) * added delete profile pic option * application period delete button red color * Roles user card scroll fix (#154) * scroll fix * added gap to user card --------- Co-authored-by: Száraz Dániel <[email protected]>
- Move author, date, and upvote field to the bottom - Delete related upvote instead of disconnecting - Update dependencies
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
- this file should not be committed into the repositoy
….com/kir-dev/schbody into feat/147-likes-on-posts
Az Eslint kicsit behalt, de szerintem azt majd megoldom egy külön pr keretében, van ott még mit configolni |
Azzal van baja az eslintnek, hogy újra van definolva a frontend config fájlban a react key a pluginokon belül. Próbáltam kiszedni az importált react objectet, kiszedni a compat.config.extends-ből is, de még mindig baja volt. |
Szerintem hivatalosan is elmondható, hogy kész a pr, maximum még az upvoteolt ikon színére valaki adhatna egy tippet, ha nem tetszik neki a kék, de az eslint is működik mostmár. |
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.
Overall great work on this! I left a few comments, but nothing groundbreaking i think.
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.
Nagyon pacek lett, beleírtam néhány dolgot, de ettől függetlenül szerintem nagyon rendben van!
}, | ||
}); | ||
|
||
const hasUpvoted = postToUpvote.upvotes.some((upvote) => upvote.userId === user.authSchId); |
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.
itt lehetne egy query-t spórolni, ha már először is upvotes.find lenne, nem?
@@ -94,7 +94,7 @@ export default function AdminApplicationPeriodCard({ period, cacheBuster, setCac | |||
}; | |||
|
|||
getProfilePicture(); | |||
}, [periodBackground, cacheBuster]); | |||
}, [periodBackground, cacheBuster, period.id]); |
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.
ez valami más miatt kellett?
return; | ||
} | ||
await api.post(`/posts/${id}/upvote`); | ||
await mutate(); |
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.
optimistic update? 🥺
alapból persze ez is tökéletes, de likeolásnál azért általában szokott lenni
A felhasználók láthatják az upvote-okat a posztokon.
A bejelentkezett felhasználók upvoteolhatják is a posztokat, és ha már upvoteolták, akkor az ikon újbóli megnyomásával levehetik az upvote-ot.
Ezenkívül létrehoztam egy AnonGuard-ot a backendben, amely mindenképp rárakja a user objektumot a requestre, de átengedi az anon felhasználókat is.
Az eddigi author és createdAt mezők a posztok aljára kerültek, de akkor is láthatóak, ha nincs lenyitva a poszt.
És a dependency-ket is frissítettem a legújabb verzióikra.
Az upvoteolt posztokon az ikon egyelőre kékre vált, de a színt bárki szabadon módosíthatja, akinek több grafikai érzéke van mint nekem.