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

Empty item list bug fix #135

Merged
merged 3 commits into from
Oct 30, 2024

Conversation

Eru111
Copy link

@Eru111 Eru111 commented Oct 30, 2024

No description provided.

Elyas Alqasim added 3 commits October 30, 2024 12:27
* 'master' of https://github.com/Eru111/tp:
  Fix incorrect variable reference
  Improve user-centric language
  Fix issues with JUnit tests
  Add table of contents and change syntax
  Change font colors in UI
  Update user guide to make it specific to our program
  Add unknown attribute test for parseContactAttribute
  Fix checkstyle issues
  Add tests for ParserUtil
  Fix test for sorting by date
  Add tests for SortCommand
  Add tests for UniquePersonList
  Fix tests for sort command parsing
  Remove unnecessary imports
  Fix ModelStub in AddCommandTest
  Fix checkstyle issues
  Remove unnecessary tests
  Add ability to sort contacts by date
  Add ascending and descending sort functionality for contacts
@Eru111 Eru111 self-assigned this Oct 30, 2024
@Eru111 Eru111 added this to the v1.5 milestone Oct 30, 2024
@Eru111 Eru111 merged commit 630e2de into AY2425S1-CS2103T-T12-3:master Oct 30, 2024
3 checks passed
Copy link

codecov bot commented Oct 30, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...java/seedu/address/logic/commands/EditCommand.java 0.00% 4 Missing ⚠️
Files with missing lines Coverage Δ Complexity Δ
...java/seedu/address/logic/commands/EditCommand.java 48.91% <0.00%> (-0.82%) 14.00 <0.00> (ø)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant