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

Oop remaster #11

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Oop remaster #11

wants to merge 13 commits into from

Conversation

ifed00
Copy link

@ifed00 ifed00 commented Oct 28, 2023

Interactions with Stepik.org moved to StepikDispatcher class, requests unified and take advantage of ?ids[] API parameter. Added unit tests (using pytest) for StepikDispatcher class.

Improvements in code readability using standard library. Added type decorations, collections.namedtuple utilized where it makes sense.

Added user reports to videos URL collection step to improve user experience.

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.

1 participant