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

Update API #132

Merged
merged 5 commits into from
Jan 24, 2020
Merged

Update API #132

merged 5 commits into from
Jan 24, 2020

Conversation

michaelarnauts
Copy link
Collaborator

Fixes #131, #129

This pull request updates the API so it works again. This caused a few changes:

  • Kids mode is now replaced by profiles
  • The API to get a single episode is gone, so some views might contain empty plots. This could be improved in the future by using get_program and filtering out the episode ourselves.

Note that these changes are still rough around the edges.

@michaelarnauts
Copy link
Collaborator Author

@dagwieers feel free to do some testing :)

@michaelarnauts
Copy link
Collaborator Author

The tests seems to be failing only for Python 2.7, but I'm not sure why.

@dagwieers
Copy link
Collaborator

Yesterday, the SD card of mij test system failed, so I need to rebuild it and I cannot find my box with micro SD cards right now. Bad timing 😕

@michaelarnauts
Copy link
Collaborator Author

Allright, all is green again. I'll merge this so master isn't broken anymore.

@michaelarnauts michaelarnauts merged commit 8386dfc into master Jan 24, 2020
@michaelarnauts michaelarnauts deleted the new-api branch January 26, 2020 12:02
@dagwieers dagwieers added this to the v0.9.3 milestone Mar 3, 2020
@dagwieers dagwieers added the enhancement New feature or request label Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTTP 426: Upgrade required
2 participants