Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update #31

Merged
merged 11 commits into from
Apr 14, 2024
Merged

Update #31

merged 11 commits into from
Apr 14, 2024

Conversation

qinxutan
Copy link
Owner

No description provided.

qinxutan and others added 11 commits April 12, 2024 12:56
* Update UI

* Change Team Size to Team Capacity

* Fix Test Case

* Edit view team command to start from 1 not 0

* fix ug

* UI Updates to show panel correctly upon command

* Further fix ui bugs

* Update UG

* Update UI.png

* Append studentid for class list students

* Update Test Class

* Retract error message changes

* Update workflow
* Update FAQ and Command summary

* Update checkstyle

* Add navigation hyperlinks to table of contents

* Update assumptions about Target Audience

* Improve formatting for UG
* Update FAQ and Command summary

* Update checkstyle

* Add navigation hyperlinks to table of contents

* Update and improve code quality for individual contribution

* Update checkstyle

---------

Co-authored-by: Jajabonks <[email protected]>
* Fix bugs

* Add Activity and Class Diagram

* Add Activity Diagram for AddStudentCommand.
* Add Class Diagram for ModuleCode.
* Update BetterModelClassDiagram based on current state of project.

* Update checkStyle

* Update Checkstyle

---------

Co-authored-by: Jajabonks <[email protected]>
Co-authored-by: Jajabonks <[email protected]>
* Reformat UG command summary and minor fixes

* Add gui section

* Fix build issue
* Add assertion test

* Check and fix UG errors
@qinxutan qinxutan merged commit 44aa1ca into qinxutan:master Apr 14, 2024
3 checks passed
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