Skip to content

Add local dashboard staging - #4777

Draft
patrick91 wants to merge 1 commit into
2026-08-14-add-dashboard-analyticsfrom
2026-08-14-add-dashboard-staging
Draft

Add local dashboard staging#4777
patrick91 wants to merge 1 commit into
2026-08-14-add-dashboard-analyticsfrom
2026-08-14-add-dashboard-staging

Conversation

@patrick91

Copy link
Copy Markdown
Member

No description provided.

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pycon Ready Ready Preview Aug 14, 2026 8:28pm

@claude

claude Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Adds a local Pretix service (Compose profile) plus a deterministic stage seeder for exercising the dashboard against realistic data, and relaxes the Pretix STORAGES setting so local dev doesn't require S3.

A couple of minor points, nothing blocking:

  • backend/dashboard/dev/seed_stage.py: Grant.objects.filter(conference=existing_conference)._raw_delete(using=Grant.objects.db) uses a private/undocumented QuerySet API (_raw_delete). Since GrantReimbursement and UserReview rows referencing these grants are already deleted above, a normal .delete() should work just as well and avoids depending on Django internals that could change across versions.
  • pretix/settings.py: STORAGES["default"] is now only overridden to S3 when AWS_STORAGE_BUCKET_NAME is set, instead of unconditionally. This is fine for local dev, but for production it now silently falls back to whatever pretix.settings defines by default if the bucket name config is ever missing/misconfigured, rather than failing loudly. Worth double-checking that's the intended behavior.

Everything else (Compose profile gating, DEBUG/hostname guard in the seeder against pointing at hosted Pretix, default local-only credentials) looks reasonable for dev-only tooling.

@patrick91
patrick91 force-pushed the 2026-08-14-add-dashboard-staging branch from 8a15f79 to 9b30104 Compare August 14, 2026 19:59
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.72%. Comparing base (5799e4f) to head (e691edb).

Additional details and impacted files
@@                         Coverage Diff                         @@
##           2026-08-14-add-dashboard-analytics    #4777   +/-   ##
===================================================================
  Coverage                               92.72%   92.72%           
===================================================================
  Files                                     364      364           
  Lines                                   11444    11444           
  Branches                                  936      936           
===================================================================
  Hits                                    10612    10612           
  Misses                                    692      692           
  Partials                                  140      140           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Shortcake-Parent: 2026-08-14-add-dashboard-analytics
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.

1 participant