Skip to content
This repository has been archived by the owner on Dec 26, 2023. It is now read-only.

Validate every usage of BeautifulSoup in UomPodcastProvider #23

Open
ed-cooper opened this issue Jan 6, 2020 · 0 comments
Open

Validate every usage of BeautifulSoup in UomPodcastProvider #23

ed-cooper opened this issue Jan 6, 2020 · 0 comments
Labels
bug Something isn't working

Comments

@ed-cooper
Copy link
Owner

Every time the .find method or equivalent is used, we should validate that html HTML item was actually found and raise a PodcastProviderError otherwise.

Currently these errors are mostly not handled and will result in confusing random exceptions.
@ed-cooper ed-cooper added bug Something isn't working help wanted Extra attention is needed labels Jan 6, 2020
ed-cooper added a commit that referenced this issue Jan 6, 2020
@ed-cooper
Remove todos, see #23
f65c45a
@ed-cooper ed-cooper removed the help wanted Extra attention is needed label May 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ed-cooper