Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

Update Alternate Contact #139

Merged

Conversation

rayshawntan
Copy link

Update AlternateContact.java regex so that the format is now AlternateContact: Name instead of AlternateContact@Name.

Update UG to include guide on formatting for alternate contacts.

This is a continuation of #124.

@rayshawntan rayshawntan added this to the v1.3b milestone Nov 1, 2023
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #139 (803be1c) into master (7a96801) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master     #139   +/-   ##
=========================================
  Coverage     72.94%   72.94%           
  Complexity      411      411           
=========================================
  Files            75       75           
  Lines          1401     1401           
  Branches        136      136           
=========================================
  Hits           1022     1022           
  Misses          337      337           
  Partials         42       42           
Files Coverage Δ
src/main/java/swe/context/logic/Messages.java 100.00% <ø> (ø)
.../swe/context/model/alternate/AlternateContact.java 66.66% <ø> (ø)
...ava/swe/context/model/util/SampleContactsUtil.java 25.00% <0.00%> (ø)

docs/UserGuide.md Outdated Show resolved Hide resolved
docs/UserGuide.md Show resolved Hide resolved
@Cloud7050 Cloud7050 mentioned this pull request Nov 1, 2023
1 task
@Cloud7050 Cloud7050 self-requested a review November 2, 2023 03:09
@bwangpj bwangpj merged commit 0a86f77 into AY2324S1-CS2103-W14-3:master Nov 2, 2023
4 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants