nusmodifications/nusmods

Allow filtering courses by Grading Basis

Closed this issue ยท 4 comments

This has historically been a very common request, and as of last year (#3427) we finally have a field from the API to definitively tell us whether a course is letter-graded or CS/CU.

Let's do it!

Relevant files:

Had a cursory look and cmiiw the gradingBasisDescription will also need to be indexed in ElasticSearch else we are not able to aggregate by that field.

https://github.com/nusmodifications/nusmods/blob/master/scrapers/nus-v2/src/services/io/elastic.ts

Yep! I'm thinking it may be easiest to just include the grading basis as an attribute of ModuleInformation

(as an aside, the different types defined in https://github.com/nusmodifications/nusmods/blob/master/scrapers/nus-v2/src/types/modules.ts should really be better-documented)

Re: documentation I think Swagger was meant to cover that at one point but it's clear that the schema there is outdated too ๐Ÿ˜… If time permits (and the project is more active again) this is a good thing to work on ๐Ÿ’ฏ

let's goooooooooooo @chew01 ๐Ÿš€๐Ÿš€๐Ÿš€