-
Notifications
You must be signed in to change notification settings - Fork 13
non-perennial plants should be automatically removed #724
Comments
see also backend/src/model/enum/life_cycle.rs to get more information about life cycle |
@tschawax can you please create an issue/PR of what exactly needs to be done in the backend? (also for #727 and so on) Ideally, it would be a PR with utopia syntax (doesn't need to compile, though) but it can also be an issue describing in prose text how the API should be changed. This issue/PR is for @horenso, so that he can quickly start (without discussions/uncertainty what is needed) once he is finished with Opensesame. |
@markus2330 I just saw that table ( What should happen then? Should I just do: pseudo code:
I don't fully understand how a set of life_times makes sense when the comments read: #[serde(rename = "annual")]
#[db_rename = "annual"]
/// on the map for current year
Annual,
#[serde(rename = "biennial")]
#[db_rename = "biennial"]
/// on the map for current and next year
Biennial,
#[serde(rename = "perennial")]
#[db_rename = "perennial"]
/// on the map until removed by user
Perennial, |
There are 275 plants that have two life_cycle entries. In these cases, use the longest provided life_cycle. i.e.:
e.g. in case of Acinos rotundifolius we use perennial, so it should not have a remove date Please update the code comments. |
Thanks for clarifying! PR is incoming shortly 🚀 |
Tasks
Use case
plant_layer
Related Pull request
No response
The text was updated successfully, but these errors were encountered: