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

Abstract web requests #21

Closed
ed-cooper opened this issue Jan 2, 2020 · 1 comment
Closed

Abstract web requests #21

ed-cooper opened this issue Jan 2, 2020 · 1 comment
Assignees
@ed-cooper
Labels
ergonomics Related to the ease of maintaining the code

Comments

@ed-cooper
Copy link
Owner

All web request logic is currently handled by __main__.py

This clutters the file, making it hard to understand and maintain

A new interface should be created for handling web requests

In addition, it should be generic, so that dependency injection can be used to assist #3
@ed-cooper ed-cooper added the ergonomics Related to the ease of maintaining the code label Jan 2, 2020
ed-cooper added a commit that referenced this issue Jan 5, 2020
@ed-cooper
Create UomPodcastProvider, begin #21
bc99add
@ed-cooper
Copy link
Owner Author

The web-refactor branch now provides equivalent functionality to the original system as of 4fd66b2

@ed-cooper ed-cooper self-assigned this Jan 5, 2020
@ed-cooper ed-cooper mentioned this issue Jan 6, 2020
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ed-cooper ed-cooper

Labels
ergonomics Related to the ease of maintaining the code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ed-cooper