Skip to content

[CS2103T-T17-3] Organ-iser#145

Open
tch1001 wants to merge 319 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-T17-3:master
Open

[CS2103T-T17-3] Organ-iser#145
tch1001 wants to merge 319 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-T17-3:master

Conversation

@tch1001

@tch1001 tch1001 commented Sep 25, 2025

Copy link
Copy Markdown

Organ-iser will help organ transplants!

baozzii and others added 29 commits October 15, 2025 17:51
Update AboutUs Roles and Responsibilities
tch1001 and others added 30 commits November 4, 2025 02:41
The quickstart section is not suitable for non technical users.

They can get confused if they don't understand bash commands.

Let's change it to a more user friendly sequence.

Still provided original cd instructions for more advanced users.
The website name is incorrect.

This can confuse users.

Let's spell organ-izer correctly.

Follows american spelling.
There are different outputs for indexes below and above the range
for the edit command.

This might confuse the user.

Let's update the command so the error is consistent.

Now it displays command usage for both.
After fixing the edit command, its tests are outdated.

This causes the app to fail deployment.

Let's update the tests so they pass.

Error messages are now uniform.
There is an unused import in edit command.

This is causing the build to fail.

Let's remove this to prevent build failures.

Single line fix.
Checkstyle errors were present in the file.

They prevented the app from building.

Let's fix those errors.

Should pass checkstyle now.
Website name is misspelled.

Can be confusing for users.

Let's fix it.

Should work now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants