prearo / ped Goto Github PK

Where to locate the empty data file (in the new system) and where to take the data file from (in the old system) - i.e. the path to the json - can be included to make the process clearer.

Overseas phone numbers containing +countrycode are not accepted.

Shows a list of all the modules and contacts in the list.

In what list is this from?

Multiple modules under the same person is unsupported.
However, in reality, some friends may share with us more than one module.

In the simplest case: CS2103T and CS2101

The usability of the app is thus affected.

The UI has a very large minimum size taking up almost all of my screen (2880x1800) and cannot be shrunk to, say, half a screen, so is quite intrusive.

When deletem or deletep is called, different invalid indices results in different error messages.

deletem 0 or deletem -1 or deletem 1.1 returns the Invalid Command Format! message

while an out of bounds index returns the invalid index message.

The latter error message could be improved.

Some modules have two different module codes but are functionally the same module, being taught as one class under the same professor, e.g. MA3238/ST3236. Other modules are cross-listed as they are taught by different departments depending on the semester, e.g. MA2116/ST2131. But these are treated as separate modules in the app.

However, this is unlikely to be much of an issue because they are preclusions anyway.

No details provided by bug reporter.

While all documentation points to the URLs provided having to be zoom links, other URLs can be added, such as
https://www.example.com
as long as they pass the regex.

(This is minor, and might be related to documentation - some modules don't use zoom)

The list of valid modules appears to not be up to date.

MA2116 Probability is not accepted but it is under its old module code of MA2216.

Given that this module is a math major core module, many people would be taking it, so this is quite a major (pun intended) issue.

The regex for the lecture/tutorial zoom links is overly strict.
Links without "https://" are rejected even though all major browsers implicitly convert them to https links if they are secure links.

While the app uses an up-to-date list of current modules, modules that are currently not offered because they have been removed or recoded are not accepted as valid input.

e.g. MA1101R Linear Algebra I was recoded to MA2001 starting from cohort 21/22 onwards, but the former is not accepted as input.

This could affect past batches trying to record down their historical modules.