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

feat: add treecluster list #38

Merged
merged 5 commits into from
Sep 15, 2024
Merged

Conversation

doriengr
Copy link
Contributor

refs #26

@doriengr doriengr self-assigned this Sep 13, 2024
@doriengr doriengr requested a review from choffmann September 13, 2024 10:51
@doriengr doriengr marked this pull request as ready for review September 13, 2024 10:51
Copy link
Member

@choffmann choffmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

{clusters.length === 0 ? (
<li className="text-center text-dark-600 mt-10">
<p>
Keine Ergebnisse mit den eingestellten Filteroptionen gefunden.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if there is no data, e.g. on first use? Then the user sees this information and may be confused?

Suggestion: Maybe check in the condition if a filter (when the filter logic is back) is active. If so, display this message, otherwise display another message.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point 👍

@@ -0,0 +1,14 @@
export enum Region {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should move this to the backend. But for now we can leave it here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add this as a note to the issue 👍

@@ -0,0 +1,14 @@
export enum WateringStatus {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, we should move this to the backend

@doriengr doriengr merged commit d36148e into develop Sep 15, 2024
1 check passed
@doriengr doriengr deleted the feature/add-treecluster-view branch September 15, 2024 05:40
@choffmann choffmann restored the feature/add-treecluster-view branch September 15, 2024 11:57
@choffmann choffmann deleted the feature/add-treecluster-view branch September 15, 2024 12:03
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

Successfully merging this pull request may close these issues.

2 participants