Tanjy55 patch 1#74
Closed
Tanjy55 wants to merge 306 commits into
Closed
Conversation
Add tests for command execution
…method and improving file path handling in LoggerConfig
Create global logger
Enhance Parser logging and update tests to use QuotelyState
Add assertion and enable assertions in tests and run configurations
…etween main and quotes
UI handling for states, and addition of NavigateCommand to navigate between main menu and quotes
update .gitignore to ignore log files and improve test coverage of pa…
Junit test for CompanyName, Item, Quote, QuoteList, QuotelyState
Updated user guide
Adding JUnit Test with assertions for NavigateCommand
Add CalculateTotalCommandTest JUnit test
V2.1 parser fixes for UI & pdf formatting issue
Added the ability to store company name fixing Issue #125
Add junit tests for naming and max values from previous PR
Modified parser to accept names with A-z and common characters only
Minor update to PPP
DG local storage
Change the filename of my PPP to lowercase as mentioned in CS2113 website
My PPP final
Final PPP
Update ppp and gson sequence diagram
Update/authorship
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.