Issue 233: Admin Quick Actions #275
Conversation
…s (working: Antrag genehmigen, Anrechenbar markieren, AEP bestanden markieren)
# Conflicts: # client/src/components/admin/UsersList.vue # client/src/utils/gateways.ts # server/src/controllers/internship.ts # server/src/routes/internship.ts
|
[del] |
| @@ -78,7 +79,7 @@ export async function getRandomInternship( | |||
| let user; | |||
| try { | |||
| user = await getUserWithInternshipModule(req.user?.email); | |||
| } catch (e: any) { | |||
| } catch (e) { | |||
There was a problem hiding this comment.
geht das jetzt doch ohne den typ any zu setzen?
There was a problem hiding this comment.
Ja, im catch-Block ist das kein Problem, da der Error ja einfach weitergegeben wird. Und das vermeidet dann, das ESLint über das any meckert :D
There was a problem hiding this comment.
Ohhh achso. Gute zu wissen!
| propsObject.supervisorEmailAddress !== undefined | ||
| ) { | ||
| internshipProps["supervisor"] = {}; | ||
| if (propsObject.supervisorFullName !== undefined) |
There was a problem hiding this comment.
reicht hier auch if (propsObject.supervisorFullName)
There was a problem hiding this comment.
auch bei den anderen undefined vergleichen. ich mein wenn's null ist ist auch doof. null braucht man nur beim patchen denk ich
There was a problem hiding this comment.
Die Methode wird ja für POST und PATCH gleichermaßen verwendet, deswegen hab ich das so gebastelt. Und im Backend dürfen die Properties ja auch auf null gesetzt werden, von daher sind's gültige Werte.
| ); | ||
|
|
||
| internshipRouter.patch( | ||
| "/:id/approve", |
There was a problem hiding this comment.
mich stört noch ein bisschen dass in der url etwas anderes als nur entities stehen (also auch ein verb) aber ich wüsste gerade auch nicht wie man's anders machen könnte. höchstens vlt `?status=approve". Hm.
There was a problem hiding this comment.
Wär ne Möglichkeit, aber dann müsste man in updateInternship wieder zwischen den einzelnen Requests unterscheiden und die entsprechende Action ausführen, anstatt auf Router-Ebene direkt auf die richtige Action zu verweisen. Ich find beides nicht optimal, aber das hier ein bisschen einfacher 😅 Deswegen würde ich es so lassen.
Fixes #233