wger icon indicating copy to clipboard operation
wger copied to clipboard

goals on nutritional plan remain set

Open Dieterbe opened this issue 1 year ago • 6 comments

Steps to Reproduce

  1. create nutritional plan
  2. under edit, "add goals to this plan"
  3. add a number under protein
  4. save
  5. edit the plan again, and UNCHECK "add goals to this plan"
  6. save again
  7. open "edit" again. "add goals to this plain" is still checked with the goal intact. it should be unchecked

i think the proper fix is, when you uncheck the "add goals", is to unset all goals fields - that's what the flutter app does.

Dieterbe avatar Apr 21 '24 20:04 Dieterbe

Hey, I would like to work on this.

IlyasOsman avatar May 16 '24 14:05 IlyasOsman

Go for it! thank you. let me know if you have further questions (you can also find me on our discord)

Dieterbe avatar May 16 '24 14:05 Dieterbe

Thank you. I'll contact you for any further questions.

IlyasOsman avatar May 16 '24 14:05 IlyasOsman

Hey, I am new to python development and thought I should work few real-time projects to get experience. can you guys guide me?

Ganesh96 avatar Jun 04 '24 03:06 Ganesh96

Hi! This would be actually a react thing since it's part of the frontend. Also, @IlyasOsman might be working on this?

rolandgeider avatar Jun 05 '24 08:06 rolandgeider

hey i would like to work on this problem!

sanjanaimandi avatar Sep 26 '24 16:09 sanjanaimandi

Hello I would love to work on this issue, if it isn't already taken by someone.

svetozar12 avatar Nov 24 '24 17:11 svetozar12

@svetozar12 go ahead!

rolandgeider avatar Nov 25 '24 09:11 rolandgeider

@svetozar12 heads up, I saw that you cloned this repo but that nutrition part is done in react: https://github.com/wger-project/react

rolandgeider avatar Nov 25 '24 15:11 rolandgeider

@rolandgeider thank you so much, I was wondering where is the react :D

svetozar12 avatar Nov 25 '24 19:11 svetozar12

one quick question, I should make my pr to https://github.com/wger-project/react right ?

svetozar12 avatar Nov 25 '24 20:11 svetozar12

yeah exactly

rolandgeider avatar Nov 25 '24 20:11 rolandgeider