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 bugs pertaining to PE-D #269

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

ethan-goh
Copy link

  • Fixed a bug where the PersonDetails panel did not clear upon calling the clear command
  • Fixed a bug where if more than one whitespace was present between modules or before modules in the EditCommand for editing modules, the command did not run

Copy link

codecov bot commented Nov 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ Complexity Δ
...ava/seedu/address/logic/commands/ClearCommand.java 100.00% <100.00%> (ø) 2.00 <0.00> (ø)
.../seedu/address/logic/parser/EditCommandParser.java 100.00% <100.00%> (ø) 17.00 <0.00> (ø)

Copy link

@JYL27 JYL27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JYL27 JYL27 merged commit 973e162 into AY2425S1-CS2103T-F15-2:master Nov 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants