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

We currently display a Recipe's date as "Brew Date", when it should probably be "Creation Date" #619

Closed
matty0ung opened this issue Sep 21, 2021 · 1 comment

Comments

@matty0ung
Copy link
Contributor

See https://bugs.launchpad.net/brewtarget/+bug/1898188 that when you copy a recipe it looks like its brew date is copied too.

Actually, in Recipe itself, the attribute is just called date and there's a comment saying The date the recipe was created or brewed. I'm not sure yet.

I suggest we either not display a Recipe's date or display/treat it as date created.

Creating a new recipe already sets its date to whatever the current date is. And we have a separate data structure (BrewNote) for recording the occasion(s) on which a recipe was brewed. So it's not like we need a recipe record to have its own brew date.

@matty0ung
Copy link
Contributor Author

Fixed in #798

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant