-
Notifications
You must be signed in to change notification settings - Fork 69
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
[v1.0][T15-B2] Smart Insurance Collection #61
base: master
Are you sure you want to change the base?
Conversation
Hi @jstarw, your pull request title is invalid. For phase A, it should be in the format of For phase B, it should be in the format of Please follow the instructions given strictly and edit your title for reprocessing. Submit only one learning outcome per pull request (unless otherwise stated in instructions) and do remember to create your branches from the commit where the Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at cs2103-pr-bot and add a link to this PR. |
sorry, i think i commited with another github account, |
should i push again with the correct account or should i just make sure any further commit is the right account @madsonic |
@yuxiangSg push it again with the correct account |
@yuxiangSg u can do it directly from your personal repo and not have to merge into the team repo if it is individual LO |
…B2/main into v1.0Yuxiang # Conflicts: # src/main/resources/view/NewMainWindow.fxml
# Conflicts: # src/main/resources/view/NewMainWindow.fxml
V1.5 Edit PersonDetail
Added some diagrams, updated my portfolio and renamed it
V1.5yuxiang sg
No description provided.