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

Parse AOI geojson to featcol in generate data extract #1983

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

Sujanadh
Copy link
Contributor

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

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

Related Issue

  • Unable to generate data extract when user draws AOI instead of uploading geojson

Describe this PR

When AOI is drawn manually, it passes geojson as:

{'type': 'Feature', 'geometry': {'type': 'Polygon', 'coordinates': [[[85.32623169895389, 27.708784900093008], [85.32627350543834, 27.706972399316967], [85.32879582999944, 27.7056900676549], [85.33067712179916, 27.707860158561616], [85.3296319596882, 27.70986992016195], [85.32766705491962, 27.710054865688036], [85.32623169895389, 27.708784900093008]]]}, 'properties': None}

It is fixed by simply parsing geojson to FeatureCollection.

Screenshots

N/A

Alternative Approaches Considered

Did you attempt any other approaches that are not documented in code?

Review Guide

Notes for the reviewer. How to test this change?

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 December 12, 2024 04:55
@Sujanadh Sujanadh self-assigned this Dec 12, 2024
@github-actions github-actions bot added bug Something isn't working backend Related to backend code labels Dec 12, 2024
@spwoodcock spwoodcock merged commit ae1b964 into development Dec 12, 2024
5 checks passed
@spwoodcock spwoodcock deleted the fix/generate-data-extract branch December 12, 2024 10:44
@spwoodcock spwoodcock added priority:high Should be addressed as a priority effort:low Likely a few hours labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code bug Something isn't working effort:low Likely a few hours priority:high Should be addressed as a priority
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants