Skip to content

GPS-898: add delete drawing feature - #12

Merged
hbollon merged 3 commits into
developfrom
GPS-898-feat-add-delete-endpoint
Aug 28, 2026
Merged

GPS-898: add delete drawing feature#12
hbollon merged 3 commits into
developfrom
GPS-898-feat-add-delete-endpoint

Conversation

@hbollon

@hbollon hbollon commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Adds the DELETE (/api/wps/v1/drawings/{drawing_id}) endpoint to permanently delete a KMZ drawing:

  • admin_id is required as a form field in the request body (aligned with the UPDATE endpoint). It is verified against the admin-id stored in S3 metadata . 403 on mismatch.
  • 404 if the drawing doesn't exist, 204 on success.
  • removes the S3 object directly (no trash folder)

This PR also includes a fix for the OPTIONS preflight crash introduced by the FastAPI upgrade in 2be6077. The crash was caused by the OpenTelemetry FastAPI instrumentation, which has been bumped to 0.65b0 to resolve it.

@hbollon
hbollon requested review from KeeTraxx and ltshb August 26, 2026 12:54
@hbollon hbollon added the feature New feature or enhancement label Aug 26, 2026
@hbollon

hbollon commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

It seems that the config for PR Labeler is missing for this repo, I will have a look

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread app/api/wps.py
@hbollon
hbollon merged commit 269891e into develop Aug 28, 2026
8 checks passed
@hbollon
hbollon deleted the GPS-898-feat-add-delete-endpoint branch August 28, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or enhancement

Development

Successfully merging this pull request may close these issues.

3 participants