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

Fix Help Window and Minor Issues #132

Merged
merged 2 commits into from
Mar 29, 2023

Conversation

eugenetangkj
Copy link

Main Changes

  • Update help window to reflect latest command information
  • Fix capitalisation for Status
  • Remove updating of right UI panel for copy and refined copied message

Issues

This PR fixes #116.

@eugenetangkj eugenetangkj marked this pull request as ready for review March 29, 2023 10:39
@eugenetangkj eugenetangkj added this to the v1.3b milestone Mar 29, 2023
@codecov
Copy link

codecov bot commented Mar 29, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.02 ⚠️

Comparison is base (6e1945a) 77.79% compared to head (b81945a) 77.77%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #132      +/-   ##
============================================
- Coverage     77.79%   77.77%   -0.02%     
  Complexity      579      579              
============================================
  Files            79       79              
  Lines          1684     1683       -1     
  Branches        188      188              
============================================
- Hits           1310     1309       -1     
  Misses          314      314              
  Partials         60       60              
Impacted Files Coverage Δ
...a/seedu/internship/logic/commands/CopyCommand.java 100.00% <ø> (ø)
...java/seedu/internship/model/internship/Status.java 93.75% <ø> (ø)
.../main/java/seedu/internship/model/InternBuddy.java 84.84% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@potty10
Copy link

potty10 commented Mar 29, 2023

LGTM!

@potty10 potty10 merged commit 718ebee into AY2223S2-CS2103T-T14-3:master Mar 29, 2023
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.

Update help window
2 participants