lesesalen / lesebot

Our reading hall needed some robotic love, this was the result
MIT License
6 stars 8 forks source link

feat: :construction_worker: Added request command #22

Closed hjalm8 closed 3 years ago

hjalm8 commented 3 years ago
* Verken `INF207` eller `INF143A` ble plassert i noen kategori, de bare havnet frittstående

category.name har funky case, burde egentlig bruke cache.find(cat => cat.id == "$ID")

* Det funket ikke å lage `DAT251` siden den ikke er i eksamensinfofila, så den `default`-branchen i `switchen` hvor du finner kategori funker ikke

Den funksjonen er kun for å hente ut hvilken kategori, i discord serveren, de skal plasseres i. Sorry, no course with the code DAT251 found... try again Betyr at den ikke fant faget i eksamensfila. User-input stringen inputSubject brukes ikke videre, og faget DAT251 vil aldri komme inn i switch casen.

Velger å ikke stole på user input. Den bruker ikke input til noe annet en å finne faget i egen database over fag.

* Når jeg prøvde uten et argument fikk jeg `Sorry, no course with the code  found... try again` som feil, tror det er fordi du gjør `if (subject == undefined)`, om du gjør `if (subject === "")` tror jeg den blir riktig. Har samme problem i f.eks. `insult` men skjønner ikke hvorfor den er ei tom streng og ikke undefined... Er kanskje mer riktig å gjøre `if (!subject)` i steden så man sjekker for begge 🤔

Hadde noen problemer med at den prøvde å kalle undefined.toUpperCase().trim(). Foreslår: if (subject === undefined || subject.trim() === ""), fordi

a = undefined;
> undefined
a === ""
> false
a == ""
> false
hjalm8 commented 3 years ago

21 Kopi

Måtte bruke git push -f et par ganger for å fikse duplicate commits

Fiksa input validation for undefined og "", la til at den peker på riktig kategori basert på channel.id, ikke navn.

Prøvde å lage en course exits funksjon, sånn at alle delene av commandoen kunne byttes ut, men det ble bare masse kluss da jeg ikke er vant til type handling i TS.