Skip to content

Conversation

@crystaltine
Copy link

No description provided.

@ganning127
Copy link
Contributor

@crystaltine Really nice start. Just a comment:

  • I recently merged a PR that allows for filtering the statistics page based on branch and status. Would you be able to make the excel export reflect the filtering? The filtering logic and data handling is done, so it should just be a case of moving the export code to the correct location.

@ganning127 ganning127 self-requested a review April 18, 2025 15:28
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 20, 2025

Deploying registration with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3430105
Status: ✅  Deploy successful!
Preview URL: https://f4838e24.registration-aft.pages.dev
Branch Preview URL: https://excel-exporter.registration-aft.pages.dev

View logs

@crystaltine
Copy link
Author

@ganning127 The exporter already uses the current filtered dataset to generate the spreadsheet - is that what you were looking for? lmk if you meant something else - i just made a merge commit and some slight improvements.

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.

3 participants