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

System health. Scrapers needed polishing #676

Merged
merged 4 commits into from Oct 30, 2022
Merged

System health. Scrapers needed polishing #676

merged 4 commits into from Oct 30, 2022

Conversation

hhursev
Copy link
Owner

@hhursev hhursev commented Oct 30, 2022

Ran sample "online" mode and addressed the issues.

  • Adds creativecanning.com, (same scraper as PracticalSelfReliance)
  • Removes sallys-blog.de. Scraper did not work and site is tricky to scrape.

Bad news is that lots of the scrapers needed polishing. Good news is (much) more sites have schema available!

- Adds creativecanning.com/ (same scraper as PracticalSelfReliance)
- Removes sallys-blog.de. Scraper did not work. Site is tricky to scrape.

Good news is that a lot of scrapers needed polishing because they have schema available!
@hhursev hhursev merged commit fa61cf2 into main Oct 30, 2022
jayaddison pushed a commit that referenced this pull request Dec 16, 2022
* Health Check. A LOT of scrapers needed polishing

- Adds creativecanning.com/ (same scraper as PracticalSelfReliance)
- Removes sallys-blog.de. Scraper did not work. Site is tricky to scrape.

Good news is that a lot of scrapers needed polishing because they have schema available!

* Post merge with main branch fixes. (needed due to design changes in sites)
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.

None yet

1 participant