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

[F10-3] Police Record System #163

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

Conversation

iamputradanish
Copy link

@fanyiii
Copy link

fanyiii commented Oct 5, 2018

To check that auto-publishing worked, https://cs2113-ay1819s1-f10-3.github.io/main/ should show your README file but currently it doesn't. (see instructions here: https://nuscs2113-ay1819s1.github.io/addressbook-level4/UsingTravis.html#enabling-auto-publishing-of-documentation)

@iamputradanish iamputradanish changed the title [F10-3] EX-SI-53 [F10-3] Police Record System Oct 11, 2018
@fanyiii
Copy link

fanyiii commented Oct 22, 2018

Good job! You can also consider using diagrams to explain how the features are implemented. For the design section, you can also considered using top-down approach, whereby the architecture diagram only includes the main component, and the low level details within each component are given in separate diagrams (e.g. by team member who's in charge of that component).

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