Skip to content

Add explorer S3 cleanup GitHub Action and clean-explorer-bucket CLI flags (#280)#333

Open
ywkim312 wants to merge 1 commit intomainfrom
280-implement-s3-bucket-cleanup-as-github-action
Open

Add explorer S3 cleanup GitHub Action and clean-explorer-bucket CLI flags (#280)#333
ywkim312 wants to merge 1 commit intomainfrom
280-implement-s3-bucket-cleanup-as-github-action

Conversation

@ywkim312
Copy link
Copy Markdown
Collaborator

@ywkim312 ywkim312 commented Mar 25, 2026

Summary

Implements issue #280: run explorer map CSV cleanup from GitHub (manual trigger) using the same rules as flask clean-explorer-bucket, and make the command safe for CI.

Changes

How to test

  1. Local (needs AWS env vars for your bucket)
    From backend/ with FLASK_APP=oeps:
    pip install -e .
    flask clean-explorer-bucket --non-interactive --dry-run

Confirm it lists Would delete … lines and exits 0.
Without credentials, expect boto/S3 errors—that only proves AWS isn’t configured.

  1. Missing sources.json + non-interactive
    Point --explorer-path at a folder with no config/sources.json:

flask clean-explorer-bucket --non-interactive --explorer-path /path/to/empty-explorer-root
Should exit with an error and not prompt.

  1. After merge
    Actions → Clean explorer S3 bucket → Run workflow; try dry run first, then a real run if the log looks correct.

Closes

Closes #280.

@ywkim312 ywkim312 linked an issue Mar 25, 2026 that may be closed by this pull request
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 25, 2026

Deploy Preview for oeps canceled.

Name Link
🔨 Latest commit 7e4fb17
🔍 Latest deploy log https://app.netlify.com/projects/oeps/deploys/69c4219cdd2ea800084fa4b8

@ywkim312 ywkim312 self-assigned this Mar 25, 2026
@ywkim312 ywkim312 requested a review from Makosak March 25, 2026 17:57
@ywkim312 ywkim312 requested a review from bodom0015 April 2, 2026 16:45
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.

Implement S3 bucket cleanup as github action

1 participant