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.

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

"Plant date" uses current date #78

Closed
9YbQiuEohUu1 opened this issue May 10, 2019 · 1 comment
Closed

"Plant date" uses current date #78

9YbQiuEohUu1 opened this issue May 10, 2019 · 1 comment
Assignees
Labels

Comments

@9YbQiuEohUu1
Copy link

When creating a new plant, the app ignores the entered date and instead uses the date when the entry is made. In the screenshot, the plant created on March 12 is displayed as planted on March 12 despite the entered plant date of January 27. The date can be changed in "Plant History" tab through "edit action" option.
Screenshot_20190327-224652_GrowTracker
Screenshot_20190327-224728_GrowTracker

@7LPdWcaW
Copy link
Owner

This looks like an unintentional consequence of the change made a while back to add in plant stages as actions.

The plant date is actually a separate property to the "planted" action, although the "planted" action is added when the plant is created, it isnt actually tied to the plant date.

It would probably make sense to always use the "planted" action date for the "plant date", but then that might cause an issue if is deleted from the history.

I'll have a think about how to best solve this

@7LPdWcaW 7LPdWcaW self-assigned this May 10, 2019
@7LPdWcaW 7LPdWcaW added the bug label May 10, 2019
7LPdWcaW added a commit that referenced this issue Aug 8, 2019
Merge branch 'feature/link-date' into develop

* feature/link-date:
  Links planted action with plant date
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants