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

Implement UI for pr and fix previous UI bugs #222

Conversation

richardcom
Copy link

Implement UI for Problem reports
Fix #207

@codecov-io
Copy link

Codecov Report

Merging #222 into master will decrease coverage by 0.60%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
- Coverage     47.17%   46.57%   -0.61%     
  Complexity      522      522              
============================================
  Files           124      126       +2     
  Lines          2476     2508      +32     
  Branches        324      329       +5     
============================================
  Hits           1168     1168              
- Misses         1223     1255      +32     
  Partials         85       85              
Impacted Files Coverage Δ Complexity Δ
...rc/main/java/seedu/address/logic/LogicManager.java 33.33% <0.00%> (-0.76%) 3.00 <0.00> (ø)
...dress/logic/commands/FindProblemReportCommand.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...edu/address/logic/commands/ViewProblemCommand.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...n/java/seedu/address/ui/ProblemReportListCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
.../java/seedu/address/ui/ProblemReportListPanel.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)

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 add17fb...a3b9585. Read the comment docs.

@davidsqf davidsqf merged commit 7824f19 into AY2021S1-CS2103-F09-3:master Nov 2, 2020
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.

[PE-D] GUI flaws
3 participants