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 UG and DG #227

Merged
merged 6 commits into from Nov 10, 2019
Merged

Conversation

jeunhoe
Copy link

@jeunhoe jeunhoe commented Nov 9, 2019

Hi all, finished up my sections of the UG and DG. Also updated most of the screenshots in UG to follow the standardised format we discussed.

…ts-branch

* 'master' of https://github.com/jeunhoe/main: (74 commits)
  Update for checkstyle
  Update Documentation
  Comply to Java Coding Standard
  Add piechart for attendance and stats for performance in CalendarDetailPanel
  Rename Attendance to TrainingManager
  updated for testcases
  Adhere to CheckStyle
  Updated for testcases
  Rename method to editPersonPerformanceRecords for consistency with editPersonTrainingRecords
  Implement methods for editing person details in events.json
  Updated to comply with checkstyle
  Updated to comply with checkstyle
  Adhere to CheckStyle
  Updated to comply with checkstyle
  Minor changes to InformationDisplay
  Rename Attendance to TrainingManager
  improve code quality
  Fix bug where double flags went unnoticed (eg. delete -e -e)
  Require non null arguments for defensive programming
  Adhere to CheckStyle
  ...
…ts-branch

* 'master' of https://github.com/jeunhoe/main: (21 commits)
  ug update
  update ug diagrams
  Update DG and UG
  update dg
  Drafted ppp
  Update UG and DG for calendar feature and sort command
  update dg
  update ug
  Update UG
  Adhere to CheckStyle
  Show "no records" message when user requests to view empty records
  Add tests for Event and Record
  Fix ScrollPane error
  Abstract out parsing of Index
  Rearrange methods for more logical flow
  Update for checkstyle
  Make language of UG more personalised
  Update UG for calendar and sort commands
  undo edit command
  Update diagrams for consistency
  ...

# Conflicts:
#	docs/DeveloperGuide.adoc
@coveralls
Copy link

Pull Request Test Coverage Report for Build 342

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 22 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.02%) to 51.978%

Files with Coverage Reduction New Missed Lines %
file:/home/travis/build/AY1920S1-CS2103T-T12-3/main/src/main/java/seedu/address/logic/commands/ViewCommand.java 3 44.44%
file:/home/travis/build/AY1920S1-CS2103T-T12-3/main/src/main/java/seedu/address/logic/commands/AttendanceCommand.java 6 0.0%
file:/home/travis/build/AY1920S1-CS2103T-T12-3/main/src/main/java/seedu/address/model/TrainingManager.java 13 64.71%
Totals Coverage Status
Change from base Build 340: 0.02%
Covered Lines: 1734
Relevant Lines: 3336

💛 - Coveralls

@hellodommy hellodommy merged commit e2f7829 into AY1920S1-CS2103T-T12-3:master Nov 10, 2019
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.

None yet

3 participants