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.

Sign up for GitHub

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 PPP #247

Merged
merged 2 commits into from
Nov 11, 2024
Merged

Update PPP #247

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ae5b421
Update PPP
cnivedit Nov 11, 2024
4c01de2
Add new sections
cnivedit Nov 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 41 additions & 6 deletions docs/team/cnivedit.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
# Chittazhi Nivedit Nandakumar - Project Portfolio Page

## Overview

Pharmacy Inventory & Logistics Ledger (PILL) is a desktop application that allows
pharmacists to keep track of and manage medicinal inventory.

Expand All @@ -17,6 +18,14 @@ Command Line Interface (CLI).
- What it does: Filters and lists expiring items based on the entered cut-off date to the user.
- Justification: Allows for easy tracking of items that are expiring post a specified cut-off date.
- Highlights: Can be accessed using the `expiring <date>` command.
- **New Feature**: Create commands and integrate order related features including placing, viewing and fulfilling
orders, viewing transactions and querying transaction history over a specified date-range
([#142](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/142)).
- What it does: Enables user to place, view and fulfill orders. Users may also view transactions and transaction
history over a specified date range.
- Justification: Allows for easy tracking of orders and transactions.
- Highlights: Replicates real life inventory functioning where the inventory is controlled by orders that are
- placed in bulk.
- **General Contributions**: Created PillLogger class ([#103](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/103)).
- What it does: Responsible for console and file logging.
- Justification: Logging is essential to understand and track the behaviour of the application.
Expand All @@ -26,33 +35,59 @@ Command Line Interface (CLI).
- Justification: Enables loading of previously saved data.
- Highlights: Loads any data on disk, skips corrupt lines.
- **New Feature**: Added ListCommand class
- What it does: The class is responsible for handling the execution of the list command ([#7](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/7)).
- What it does: The class is responsible for handling the execution of the list
command ([#7](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/7)).
Comment on lines +38 to +39
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wow, I remember when the numbers were single digits. How time flies

- Justification: Abstraction of the list command, inheriting from the base command class.
- Highlights: Receives ItemMap instance and lists items.
- **New Feature**: Added FindItemCommand class
- What it does: The class is responsible for handling the execution of the find command ([#20](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/20)).
- What it does: The class is responsible for handling the execution of the find
command ([#20](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/20)).
- Justification: Abstraction of the find command, inheriting from the base command class.
- Highlights: Receives item name to search for and executes the command.
- **Code contributed**: [RepoSense link](https://nus-cs2113-ay2425s1.github.io/tp-dashboard/?search=cnivedit&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=&groupSelect=groupByRepos&breakdown=true&checkedFileTypes=docs~functional-code~test-code~other&since=2024-09-20&tabOpen=true&tabType=authorship&tabAuthor=yijiano&tabRepo=AY2425S1-CS2113-W14-4%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=false&authorshipFileTypes=docs~functional-code~test-code~other&authorshipIsBinaryFileTypeChecked=false&authorshipIsIgnoredFilesChecked=false)
- **Code contributed
**: [RepoSense link](https://nus-cs2113-ay2425s1.github.io/tp-dashboard/?search=cnivedit&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=&groupSelect=groupByRepos&breakdown=true&checkedFileTypes=docs~functional-code~test-code~other&since=2024-09-20&tabOpen=true&tabType=authorship&tabAuthor=yijiano&tabRepo=AY2425S1-CS2113-W14-4%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=false&authorshipFileTypes=docs~functional-code~test-code~other&authorshipIsBinaryFileTypeChecked=false&authorshipIsIgnoredFilesChecked=false)
- **Project Management**:
- Helped maintain some issues, opening
[#69](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/69),
[#71](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/71),

[#143](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/143),
[#145](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/145),
[#146](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/146),
[#163](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/163),
[#239](https://github.com/AY2425S1-CS2113-W14-4/tp/issues/239),

- **Testing**:
- Commands
- ListCommand ([#49](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/49))
- FindCommand ([#51](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/51))
- Methods
- getExpiringItems ([#113](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/113))
- loadLine ([#68](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/68))

- **Documentation**:
- Developer Guide
- Added the following sections
- AddItem command sequence diagram ([#124](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/124)):
- Orders and Transactions ([#241](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/241))
- AddItem command sequence diagram ([#124](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/124))
- Logging ([#121](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/121))
- Target User Profile ([#115](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/115))
- Value Proposition ([#115](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/115))
- User Stories ([#115](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/115))
- User Guide
- Fixed inaccurate order command format ([#224](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/224))
- Update fulfill command usage ([#235](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/235))

- **Community**:
- PRs reviewed(with non-trivial comments):
[#56](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/56),
[#72](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/72),
[#106](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/106),
[#122](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/122),
[#125](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/125)
[#125](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/125),
[#164](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/164),
[#168](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/168),
[#223](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/223),
[#232](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/232),
[#233](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/233),
[#240](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/240),
[#243](https://github.com/AY2425S1-CS2113-W14-4/tp/pull/243)
Loading