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.

Sign up for GitHub

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

Jump to bottom

[PE-D] Target weight with decimal points not accepted #185

Closed
nus-pe-bot opened this issue Oct 30, 2020 · 0 comments · Fixed by #212
Closed

[PE-D] Target weight with decimal points not accepted #185

nus-pe-bot opened this issue Oct 30, 2020 · 0 comments · Fixed by #212
Assignees
@wayne987
Labels
bug Something isn't working
Milestone
v1.4

Comments

@nus-pe-bot
Copy link

No details provided.
image.png

May be a rather inflexible feature because realistically users may have weight goals in smaller increments (e.g. increments of 0.5kg)

Labels: severity.Medium type.FeatureFlaw
original: nicholas-gcc/ped#15
@wayne987 wayne987 self-assigned this Oct 30, 2020
@wayne987 wayne987 added the bug Something isn't working label Oct 30, 2020
@wayne987 wayne987 added this to the v1.4 milestone Oct 30, 2020
@wayne987 wayne987 linked a pull request Oct 30, 2020 that will close this issue
Fix bug #212
Merged
wayne987 pushed a commit to wayne987/tp that referenced this issue Oct 30, 2020
[PE-D] Target weight with decimal points not accepted AY2021S1-CS2103…
94dabef 
…T-W11-3#185
wayne987 pushed a commit to wayne987/tp that referenced this issue Oct 31, 2020
Merge branch 'master' into Add-Profile-Picture
814c395 
* master:
  fix bug where you can change a profile id to an existing profile with the same id
  fix test case
  Create command can take in multiple name (n/) fields AY2021S1-CS2103T-W11-3#173
  [PE-D] Cannot add multiple foods in the same timeslot AY2021S1-CS2103T-W11-3#181
  [PE-D] Undescriptive error for negative calories AY2021S1-CS2103T-W11-3#179
  [PE-D] Target weight with decimal points not accepted AY2021S1-CS2103T-W11-3#185
  [PE-D] Duplicate profile ID flaw incompletely handled AY2021S1-CS2103T-W11-3#186
  [PE-D] Unchecked Date for add command AY2021S1-CS2103T-W11-3#191
  [PE-D] Error Message is vague AY2021S1-CS2103T-W11-3#199
  [PE-D] `create` command example not valid AY2021S1-CS2103T-W11-3#206
  fix a bug if the user clear and create a profile with the same field as the default profile, he would'nt be able to add day
  set a better valid range for profile height
  Bug fix : AY2021S1-CS2103T-W11-3#210 Java Integer Overflow
  Fix multiple bugs for UI
wayne987 pushed a commit to wayne987/tp that referenced this issue Nov 1, 2020
Merge branch 'master' into progress-bar
54b1620 
* master:
  fix bug where you can change a profile id to an existing profile with the same id
  fix test case
  Create command can take in multiple name (n/) fields AY2021S1-CS2103T-W11-3#173
  [PE-D] Cannot add multiple foods in the same timeslot AY2021S1-CS2103T-W11-3#181
  [PE-D] Undescriptive error for negative calories AY2021S1-CS2103T-W11-3#179
  [PE-D] Target weight with decimal points not accepted AY2021S1-CS2103T-W11-3#185
  [PE-D] Duplicate profile ID flaw incompletely handled AY2021S1-CS2103T-W11-3#186
  [PE-D] Unchecked Date for add command AY2021S1-CS2103T-W11-3#191
  [PE-D] Error Message is vague AY2021S1-CS2103T-W11-3#199
  [PE-D] `create` command example not valid AY2021S1-CS2103T-W11-3#206
  fix a bug if the user clear and create a profile with the same field as the default profile, he would'nt be able to add day
  set a better valid range for profile height
  Bug fix : AY2021S1-CS2103T-W11-3#210 Java Integer Overflow
  Fix multiple bugs for UI

# Conflicts:
#	src/main/java/seedu/address/model/person/Person.java
#	src/main/java/seedu/address/model/person/Profile.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wayne987 wayne987

Labels
bug Something isn't working
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

Fix bug wayne987/tp
2 participants
@nus-pe-bot @wayne987