Feat/booking checkbox - Githubissues

MindChirp commented 6 months ago

Implemented the following features:

Redesigned the booking page to make it look sleeker
Added a "Jeg godtar vilkårene for bruk og utlån av TIHLDEs eiendeler" checkbox.
Added support for creating reservation requests on behalf of groups which the user is a member of (this involves fetching memberships from backend. In other words, I have added some api functions and types).
Added a loading spinner for the submit button, but it does not show because the onSubmit function does not return a promise. This has to be fixed in the future, as I couldn't be bothered right now 🥸.
Made the "Sender inn forespørsel som" box react to the selected group.
Important note: We have to add support for sending requests on behalf of groups in backend. This is a nice little project for @eriskjel. ✅Fixed

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
kont-res	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 23, 2024 4:31pm

eriskjel commented 6 months ago

kan du fille meg litt inn på hva som legges i å sende på vegne av en gruppe? hva er hensikten og hva gir det

eriskjel commented 6 months ago

jeg er med hvis dette innebærer at man kan velge om man vil sende som enkeltperson eller på vegne av sin gruppe?

eriskjel commented 6 months ago

og jeg får ikke logga inn og testa

MindChirp commented 6 months ago

@eriskjel Ja, det er at man skal kunne sende inn en forespørsel på vegne av en gruppe. Eksempelvis vil hovedstyret kunne sende inn en søknad om å reservere kontoret, og da kan et medlem av hovedstyret sende inn søknaden på vegne av dem. Det eneste du må implementere er et felt i modellen for reservasjon som du kan kalle for group eller lignende, så sender frontend gruppeid-en i post-requesten

MindChirp commented 6 months ago

og jeg får ikke logga inn og testa

Du må logge inn med index som brukernavn og index123 som passord

MindChirp commented 6 months ago

Du burde også legge til validation som sjekker at brukeren som sender inn forspørselen faktisk har medlemskap i den oppgitte gruppa (dersom den er oppgitt)

eriskjel commented 6 months ago

og med det kan du anse gruppelogikken i Lepton som implementert.

MindChirp commented 6 months ago

@ConradOsvik eller @Embretr kan en av dere ta en titt på PRen? 🥺

eriskjel commented 6 months ago

et par conflicts her da

MindChirp commented 6 months ago

et par conflicts her da

lover, alt er under kontroll 😅

ConradOsvik commented 6 months ago

Ser ut som det er merge conflicts, @MindChirp vil du se på disse?

MindChirp commented 6 months ago

Ser bra ut! Er en stund siden så husker ikke helt nøyaktig, kanskje @Embretr husker bedre, men de ville at vi skulle ha en link til denne siden i tilfelle av alkohol på kontoret: https://hjelp.ntnu.no/tas/public/ssp/content/serviceflow?unid=8f090c9e58444762876750db1104178d&from=aef98c8c-3eb9-4e29-8439-e79834d88223&openedFromService=true

det er snart 2 uker siden så husker lite annet enn at de ville ha dette på siden som man booker kontoret på. Godkjenner pr-en men hvis vi ikke har dette, burde vi kanskje fikset det før/på mandag?

Jeg kan undersøke det!

MindChirp commented 6 months ago

Ser ut som det er merge conflicts, @MindChirp vil du se på disse?

Jobber på saken 🫡

Embretr commented 6 months ago

Må ha en avhuking på at den der er lest gjennom og godkjent før folk får booke kontoret for aøkohol

Sendt fra Outlook for iOShttps://aka.ms/o0ukef

Fra: Frikk Balder @.> Sendt: Friday, February 23, 2024 4:55:48 PM Til: TIHLDE/kontresv2 @.> Kopi: Embret Roås @.>; Mention @.> Emne: Re: [TIHLDE/kontresv2] Feat/booking checkbox (PR #47)

Ser ut som det er merge conflicts, @MindChirphttps://github.com/MindChirp vil du se på disse?

Jobber på saken 🫡

— Reply to this email directly, view it on GitHubhttps://github.com/TIHLDE/kontresv2/pull/47#issuecomment-1961579435, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQOMVFNI6U7GBTTOHGMDQTYVC3YJAVCNFSM6AAAAABDSBMSVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRRGU3TSNBTGU. You are receiving this because you were mentioned.Message ID: @.***>

MindChirp commented 6 months ago

Må ha en avhuking på at den der er lest gjennom og godkjent før folk får booke kontoret for aøkohol Sendt fra Outlook for iOShttps://aka.ms/o0ukef … ____ Fra: Frikk Balder @.> Sendt: Friday, February 23, 2024 4:55:48 PM Til: TIHLDE/kontresv2 @.> Kopi: Embret Roås @.>; Mention @.> Emne: Re: [TIHLDE/kontresv2] Feat/booking checkbox (PR #47) Ser ut som det er merge conflicts, @MindChirphttps://github.com/MindChirp vil du se på disse? Jobber på saken 🫡 — Reply to this email directly, view it on GitHub<#47 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQOMVFNI6U7GBTTOHGMDQTYVC3YJAVCNFSM6AAAAABDSBMSVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRRGU3TSNBTGU. You are receiving this because you were mentioned.Message ID: @.***>

Ait, skal fikse det

MindChirp commented 6 months ago

@Embretr @ConradOsvik Jeg fikser det i en annen branch. Jeg merger denne om det går fint for dere

TIHLDE / kontresv2

Feat/booking checkbox #47