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

[v1.2][F14-B3] Investigapptor #70

Open
wants to merge 733 commits into
base: master
Choose a base branch
from

Conversation

quentinkhoo
Copy link

No description provided.

Copy link

@liuyang1213 liuyang1213 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome

@quentinkhoo quentinkhoo changed the title [v1.0][F14-B3] Investigapptor [v1.1][F14-B3] Investigapptor Mar 16, 2018
@quentinkhoo quentinkhoo changed the title [v1.1][F14-B3] Investigapptor [v1.0][F14-B3] Investigapptor Mar 16, 2018
@quentinkhoo quentinkhoo changed the title [v1.0][F14-B3] Investigapptor [v1.1][F14-B3] Investigapptor Mar 16, 2018
@quentinkhoo quentinkhoo changed the title [v1.1][F14-B3] Investigapptor [v1.2][F14-B3] Investigapptor Mar 23, 2018
wynonaK pushed a commit to wynonaK/addressbook-level4 that referenced this pull request Mar 26, 2018
@liuyang1213
Copy link

Very thorough documentation of the new feathers, great! One suggestion is that you could use diagrams/examples to describe how a feature is implemented. Use of code snippets is good but should be minimal since code gets outdated very quickly. Try to draw some UML diagrams we learn so far and I believe that you are able to do that well 👍

quentinkhoo and others added 30 commits April 15, 2018 20:47
Update DG (cases section) + Portfolio
Update PPP to fit page limit
Update CrimeCase start date validation to accept today's date + Collate
Update UserGuide with minor tip for password
Correct AboutUs reference to PPP
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