Skip to content
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

Add arun.gov.uk source #2380

Merged
merged 2 commits into from
Aug 5, 2024
Merged

Add arun.gov.uk source #2380

merged 2 commits into from
Aug 5, 2024

Conversation

whi-tw
Copy link
Contributor

@whi-tw whi-tw commented Aug 5, 2024

BS4 parser required as Arun doesn't support ICS.

Have to set up a session initially by sending GET requests to two URLs to:

  1. Start the session
  2. Activate the session

This is quite brittle, but I guess so are most of these form scrapers!

@5ila5 5ila5 merged commit e9716dc into mampfes:master Aug 5, 2024
2 checks passed
@5ila5
Copy link
Collaborator

5ila5 commented Aug 5, 2024

Thanks for your contribution

Did some minor reformatting (.pre-commit-config.yaml)

@whi-tw
Copy link
Contributor Author

whi-tw commented Aug 5, 2024

Makes perfect sense - sorry, I didn't switch it on apparently. nor did I sign my commit. what a scrub! Thanks for just sorting it :)

@whi-tw whi-tw deleted the add_arun_gov_uk branch August 5, 2024 13:48
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.

2 participants