Skip to content

Use CFApp /api/v1/commitfest/active endpoint to find CFIds #26

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

polobo
Copy link

@polobo polobo commented Apr 14, 2025

This will need to be applied after the corresponding CFApp commit that exposes the API.

This is only lightly tested. I wanted to get something out since I've pushed a PR for the CFApp side of this.

I did quite a bit more API-based work but have yet to get a running instance using the scraping against a dev CFApp in place. Assuming the replaced code worked switching from the "+1" computation of a CF to getting three explicit CF IDs from the API should not impact any later code in the processing stream.

I made an educated guess on the rebuild/build_page code. Putting out links to potentially non-existent URLs (the In Progress one will not be active for like half the year) isn't ideal but given the user base and non-primary nature of this UI the expedient solution seems reasonable.

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