Skip to content

add citizen_science scraper for taxila#1239

Merged
fbacall merged 2 commits into
ElixirTeSS:masterfrom
DaanVanVugt:feature/taxila_citizen_science_scraper
Mar 6, 2026
Merged

add citizen_science scraper for taxila#1239
fbacall merged 2 commits into
ElixirTeSS:masterfrom
DaanVanVugt:feature/taxila_citizen_science_scraper

Conversation

@mikesndrs

@mikesndrs mikesndrs commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree
    to license it to the TeSS codebase under the
    BSD license.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new Citizen Science web scraper (ingestor) for the Taxila framework that scrapes both events and materials from citizenscience.nl. The ingestor parses HTML pages to extract event data (including Dutch month translation) and material data (from resources and training resources pages), and is registered in the ingestor factory.

Changes:

  • New CitizenScienceIngestor class that scrapes events from citizenscience.nl/events/ and materials from /resources and /training_resources pages
  • Test file covering both event and material ingestion with VCR cassettes
  • Registration of the new ingestor in the IngestorFactory

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
lib/ingestors/taxila/citizen_science_ingestor.rb New ingestor that scrapes citizen science events and materials from citizenscience.nl, with Dutch-to-English month mapping
test/unit/ingestors/taxila/citizen_science_ingestor_test.rb Tests for both event ingestion (13 events) and material ingestion (25 new + 8 updated materials)
lib/ingestors/ingestor_factory.rb Adds CitizenScienceIngestor to the Taxila ingestors list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread lib/ingestors/taxila/citizen_science_ingestor.rb Outdated
Comment thread lib/ingestors/taxila/citizen_science_ingestor.rb
Comment thread test/unit/ingestors/taxila/citizen_science_ingestor_test.rb Outdated
Comment thread test/unit/ingestors/taxila/citizen_science_ingestor_test.rb Outdated
@fbacall
fbacall merged commit bb37024 into ElixirTeSS:master Mar 6, 2026
7 checks passed
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