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

Resolved issues on project summaries with no centroids #1359

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

Sujanadh
Copy link
Contributor

@Sujanadh Sujanadh commented Mar 18, 2024

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

Describe this PR

When the project is created without boundary , it fails the /summaries endpoint since it requires centroid for clustering.
This PR handles the null case of centroid in the schema.

Screenshots

N/A

Review Guide

N/A

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@Sujanadh Sujanadh requested a review from spwoodcock March 18, 2024 06:20
@Sujanadh Sujanadh self-assigned this Mar 18, 2024
@github-actions github-actions bot added frontend Related to frontend code backend Related to backend code labels Mar 18, 2024
@varun2948 varun2948 merged commit 1fc70c7 into development Mar 18, 2024
6 checks passed
@spwoodcock spwoodcock deleted the fix/project-summaries branch March 18, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code frontend Related to frontend code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

the project created without AOI causing it to crash during validation
2 participants