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

Jump to bottom

[CS2103T-T11-2] AgenSee #65

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

[CS2103T-T11-2] AgenSee #65

wants to merge 714 commits into from

Conversation

WJunHong
Copy link

This is the main PR for CS2103T-T11-2's master branch into AY2122S2/tp master branch

Team members:

  • Wang Jun Hong
  • Janald Ho
  • Bian Yuqi
  • Chua Shi Hong
  • Phua Jun Heng

@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2022 •
edited
Loading

Codecov Report

Merging #65 (fa0004c) into master (4f7660a) will decrease coverage by 1.14%.
The diff coverage is 75.80%.

@@             Coverage Diff              @@
##             master      #65      +/-   ##
============================================
- Coverage     72.15%   71.01%   -1.15%     
- Complexity      399      879     +480     
============================================
  Files            70      141      +71     
  Lines          1232     2712    +1480     
  Branches        125      339     +214     
============================================
+ Hits            889     1926    +1037     
- Misses          311      644     +333     
- Partials         32      142     +110
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
src/main/java/seedu/address/model/client/Name.java 80.00% <ø> (ø)
...rc/main/java/seedu/address/model/client/Phone.java 80.00% <ø> (ø)
...s/model/client/PhoneContainsKeywordsPredicate.java 0.00% <0.00%> (ø)
...a/seedu/address/model/client/UniqueClientList.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/model/tag/Tag.java 80.00% <ø> (ø)
.../java/seedu/address/storage/JsonAdaptedClient.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/BuyerCard.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/BuyerListPanel.java 0.00% <0.00%> (ø)
... and 113 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f7660a...fa0004c. Read the comment docs.

janald99 and others added 29 commits April 6, 2022 18:32
@janald99
Remove "any" from house type so that there is no more confusion
271e838
@janald99
Merge branch 'master' into branch-bug-fix
ee419f3
@janald99
Fix table of content for sorting buyers
9689eb3
@janald99
Fix UG bug at find buyers where there is additional markdown
33931fc
@janald99
Merge branch 'branch-bug-fix' of https://github.com/janald99/tp into …
6feba9e 
…branch-bug-fix
@WJunHong
Added some extra notes to UG
3e00dde 
Before, no explanation for weird inputs.

Now, we tell them how they will be interpreted. Also, update the index
styling.
@janald99
Update PPP
d9b12a3
@WJunHong
Update indexing error message
a207461 
Before, it wasn't clear what was an invalid index.

Now, it is clearer.
@WJunHong
Fix error messages for delete and add property commands
3070785 
Before, some error messages were not formatted properly, and not clear
enough.

Now, they are clear and to the point.
@WJunHong
Fix test case
b8a6f9b
@janald99
Merge pull request #267 from WJunHong/branch-Fix-UG2
461257e 
More UG explanations, fix error messages
@WJunHong
Update PPP
7fa183f
@cwnm
Merge pull request #268 from WJunHong/branch-Fix-UG2
f15fdff 
Update PPP Jun Hong
@janald99
Add HouseBuilder class
85f56d2
@janald99
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp …
b412850 
…into branch-bug-fix

# Conflicts:
#	src/main/java/seedu/address/commons/core/Messages.java
@janald99
Change message displayed for the match commands to buyers instead of
073e2b4 
clients

Conform to checkstyle
@janald99
Merge pull request #266 from janald99/branch-bug-fix
ab12f57 
Branch bug fix
@janald99
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
4a301ed
@janald99
Add Property Builders and House Builder classes
23f687a
@janald99
Add test cases for Address Class
7cdf9ba
@janald99
Add test cases for HouseTypeMatchBuyerPredicateTest class
768d86e
@janald99
Add LocationMatchBuyerPredicateTest class and conform to checkstyle
23c2a4d
@janald99
Add test cases for PriceRangeMatchBuyerPredicate class
47b69f9
@janald99
Fix test case for PriceRangeMatchBuyerPredicateTest
e9005f0
@Jacky142857
Update Appointment Command to handle empty input after 'time/' prefix
9567184
@Jacky142857
Remove optional bracket in User Guide
5fc5e0c
@Jacky142857
Fix boundary dates
b19b55c
@Jacky142857
Fix checkstyle
e596472
@WJunHong
Merge pull request #270 from Jacky142857/branch-bug-fix-1
e2e5336 
Branch bug fix 1
WJunHong and others added 30 commits April 11, 2022 21:31
@WJunHong
Update sequence diagram
1de5e51
@WJunHong
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
9ff182d
@chuashihong
update DG about Edit Buyer feature and Saving data section's format e…
06d0f49 
…rror
@chuashihong
Merge pull request #342 from WJunHong/master
a922991 
Update DG bugs
@WJunHong
Merge pull request #343 from chuashihong/branch-final-PPP
a38c621 
update DG about Edit Buyer feature, Saving data section's format error
@WJunHong
Fix spelling
43477aa
@WJunHong
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
6f7255c
@chuashihong
Merge pull request #344 from WJunHong/master
f0a6c02 
Fix the DG enum
@WJunHong
Update DG
14d516e
@WJunHong
Add tags
080fe5f
@WJunHong
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
7166d08
@WJunHong
Change Command summary
418d6db
@chuashihong
Merge pull request #346 from WJunHong/master
efe94af 
Fix DG final
@WJunHong
Fix ToC for DG
982c480
@WJunHong
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
4d26551
@janald99
Merge pull request #347 from WJunHong/master
07630be 
Fix ToC for DG
@WJunHong
Fix DG AGAIN
3d81b40
@Jacky142857
Update dg
6e2f2bb
@WJunHong
Fix spacing
3a39468
@WJunHong
Merge branch 'master' of https://github.com/AY2122S2-CS2103T-T11-2/tp
543ffa2
@Jacky142857
Dummy
12d7122
@Jacky142857
dummy1
8f84be3
@Jacky142857
Merge pull request #349 from Jacky142857/branch-appt-dg
80a993f 
Update dg
@WJunHong
Fix merge conflict
fe77da5
@janald99
Merge pull request #350 from WJunHong/branch-BackUp
fa0004c 
Branch back up
@WJunHong
Final commit
57f0486
@Jacky142857
Update ppp
a9c35b0
@Jacky142857
Merge branch 'master' of https://github.com/Jacky142857/tp into branc…
02a124a 
…h-1052-ppp
@janald99
Merge pull request #351 from WJunHong/branch-BackUp
2e1a166 
Final commit
@janald99
Merge pull request #352 from Jacky142857/branch-1052-ppp
0760849 
Branch 1052 ppp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@WJunHong @codecov-commenter @janald99 @cwnm @Jacky142857 @chuashihong