Skip to content
This repository has been archived by the owner on Jul 9, 2022. It is now read-only.

Conti: scraper fixed (#73) #90

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

biligonzales
Copy link
Contributor

Describe the changes

Fixed the Conti scraper to use the newsList javascript item because no html elements were available any longer.

Related issue(s)

This fixes issue #73

How was it tested?

  1. Add Conti url to config.yaml
  2. Run docker-compose build app
  3. Run docker-compose up --abort-on-container-exit
  4. Conti results are pushed again in the database

Checklist for a new scraper (delete if N/A)

  • The URL for the site is nowhere in the git history
  • The site is added to config.sample.yaml
  • There aren't any debug logging statements/etc. (there was one logging.debug there, I left it as it was)
  • The data going into the DB is properly parsed and is accurate

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant