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

Person: update result message #1027

Closed
wants to merge 2 commits into from

Conversation

jonahtanjz
Copy link
Contributor

Fixes #453.

Update result message with ";" as delimiters and omit "Tags:" if it is not present.

There are no delimiters between each field entered and an additional
phrase 'Tags:' is always present even if the user did not input any
tags.

As a result, the result message is not as user-friendly.

Let's add delimiters and remove 'Tags:' from the result message if tags
are not present.
@canihasreview
Copy link

canihasreview bot commented Dec 7, 2020

Click here to submit a new iteration when this PR is ready for review.

See this repository's contribution guide for more information.

@jonahtanjz jonahtanjz closed this Dec 7, 2020
@jonahtanjz jonahtanjz deleted the branch-result-message branch December 7, 2020 05:10
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.

Result message can be more user-friendly
1 participant