{"push_id":2952871933,"size":1,"distinct_size":1,"ref":"refs/heads/Linh_Chi","head":"803b1de792f2cafc344ecbb0b2bb77a155d82051","before":"5f0830c83867d0cc553454d5751137f24d5b8cae","commits":[{"sha":"803b1de792f2cafc344ecbb0b2bb77a155d82051","author":{"name":"NGUYEN NGOC LINH CHI","email":"05dbf1fa88f24a72631b42dc9bea8f69f5d0026e@users.noreply.github.com"},"message":"Ignore Summary Command (#52)\n\n* Try to add the limit command\r\n\r\n* The dummy command working, the program can recognize the command but the command will do nothing at all.\r\n\r\n* Some slight changes to the program, money and limit model added, some bugs fixed.\r\n\r\n* Delete a dummy limit command\r\n\r\n* Working version with the limit command just return the failed infomation.\r\n\r\n* Working limit command praser, which can tell the limit command but can not read the correct form of the command\r\n\r\n* Working version with the limit command can be recognized but will report the incorrect format and show the information accordingly. New model \"Money\" added.\r\n\r\n* minor changes to pass the tests.\r\n\r\n* Added a new java fxml file for detailed panel UI\r\n\r\n* Hooked up the new detailed Record panel with the existing UI\r\n\r\n* The LimitCommandParser is working, the command can be recognized when the format is correct.\r\n\r\n* Updated DetailedRecordCard.java to follow new package name\r\n\r\n* Fixed sizing errors with new UI\r\n\r\n* Configured detail card to disappear when no record card is selected\r\n\r\n* Some minor changes to better merge with the FP version\r\n\r\n* Delete the class Money, the program and limit command still working.\r\n\r\n* Update the Check style in 4 files\r\n\r\n* Added DetailedRecordCardHandle as a handle to DetailedRecordCard\r\n\r\n* Fixed some JUnit tests by linking up DetailedRecordCardHandle with FinancialPlannerSystemTest\r\n\r\n* Changed tests in help system test to allow help window to be open when text box of DetailedCard is clicked\r\n\r\n* Removed BrowserPanelHandle, BrowserPanel, BrowserPanelTest\r\n\r\n* Fixed checkstyle errors #1\r\n\r\n* Fixed checkstyle errors #2\r\n\r\n* Some minor changes\r\n\r\n* The limit Command now can calculate the sum of spend during the date range.\r\n\r\n* Problem noticed, the read in value date always read in the same date for both date start and date end\r\n\r\n* minor changes to make the logic of limit correct.\r\n\r\n* The LimitCommandPhaser bug fixed. Finally the LimitCommand (Simple version), is working!!(with many check style errors tho)\r\n\r\n* Some minor changes to adjust the current working version.\r\n\r\n* Todo list updated\r\n\r\n* limit command updated, the minus bugs fixed. Now the input of money can only be double number without sign in front. && The same amount of money will not regard as limit exceeded.\r\n\r\n* some minor changes about the checkstyle\r\n\r\n* more minor changes to pass the checkstyle test\r\n\r\n* more minor changes about the checkstyle.\r\n\r\n* Some minor changes about the output.\r\n\r\n* more minor changes to pass the checkstyle test\r\n\r\n* minor changes again about the checkstyle errors.\r\n\r\n* more changes to pass the test again :(\r\n\r\n* more minor changes again.....\r\n\r\n* Modify the DeletebyDateEntryTest and add DateEqualPredicate\r\n\r\n* Modify checkstyle\r\n\r\n* Modify tests - final version\r\n\r\n* Removed codacy badge from README\r\n\r\n* Refactored the storage to make it more modular\r\n\r\n* Modify the DateIsWithinIntervalPredicate, add DateIsWithinIntervalPredicatetest, resolve the conflict\r\n\r\n* Modify the Financial Planner to ignore Summary Command\r\n\r\n* Modify DateIsWithinIOntervalPredicate and add DateIsWithinIOntervalPredicateTest\r\n\r\n* Ignore Summary Command v1.1\r\n\r\n* Fix Checkstyle","distinct":true,"url":"https://api.github.com/repos/CS2113-AY1819S1-T09-4/main/commits/803b1de792f2cafc344ecbb0b2bb77a155d82051"}]}
{"actor":{"display_login":"nguyenngoclinhchi"}}