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

[T16-4] Centralised Human Resource System #60

Open
wants to merge 1,617 commits into
base: master
Choose a base branch
from

Conversation

LimYiSheng
Copy link

@LimYiSheng LimYiSheng commented Oct 8, 2018

Recreation of PR due to Travis-CI bug
@zhihong8888 @ChuaZhenWei @ryanchen2018 @XiiaoPanda

v1.1

Updated

  • Organization setup/Project management:
    • Setup the team organization appropriately
    • Setup your project repo main
    • Enabled the issue tracker for your project repo
    • Setup auto-publishing
    • Create PRs for each team member and merge the code to the master branch
    • git tag your project code (as v1.1) containing each team member's contribution
  • Documentation:
    • Ported the documentation (user guide and developer guide) to adoc format in the project repo
    • Updated the README.adoc to reflect our project
    • Updated the About Us page with our mugshots
  • Product:
    • Merged each team member's code to our project's master branch

Mid v1.1

Updated

  • README.adoc
  • AboutUs.adoc

@jordancjq
Copy link

@LimYiSheng @zhihong8888 @ChuaZhenWei @ryanchen2018 @XiiaoPanda

Overall good effort for your documentation in v1.2! 👍

Developer Guide

  • Remember to update your component diagrams to reflect the new changes implemented
  • Good job on having a writeup for each individual implementation
  • Try to include a sequence diagram for each implementation to illustrate how the implementation work
  • Do not rely too much on code snippets as code changes quickly
  • Section 3.4 Class Diagram: as mentioned in tutorial, what would happen if you have association lines and the classes in another class?

ryanchen2018 and others added 30 commits November 12, 2018 18:28
Updated User and Developer Guide
V1.4.5 Update Developer guide and PPP
Updated prerequisite for modifyPay and modifyAllPay manual testing
Update on develop guide and user guide
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants