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

Rework Kids zone #67

Merged
merged 1 commit into from
Sep 13, 2019
Merged

Rework Kids zone #67

merged 1 commit into from
Sep 13, 2019

Conversation

dagwieers
Copy link
Collaborator

@dagwieers dagwieers commented Sep 13, 2019

This makes the Kids zone part of the routing.

@dagwieers dagwieers added the enhancement New feature or request label Sep 13, 2019
@dagwieers dagwieers added this to the v0.9.0 milestone Sep 13, 2019
@dagwieers
Copy link
Collaborator Author

dagwieers commented Sep 13, 2019

This needs a bit more work, I would rather have a single setting:

  • Kids zone: [ Disable | Enable | Force ]

Rather than:

  • Enable Kids Zone: [ True | False ]
    • Force Kids Zone: [ True | False ]

@dagwieers
Copy link
Collaborator Author

I also need to investigate if Kids zone has its own search functionality.

And for the breadcrumbs (that are now implemented for Kids zone) it is not inherited to routes that don't have a kids-variant (like /program or /movie). Not sure if that is needed or not.

@michaelarnauts
Copy link
Collaborator

Kids zone does have a different search endpoint. The results are filtered.

@dagwieers dagwieers force-pushed the kidszone branch 2 times, most recently from 659e542 to 42dd7a9 Compare September 13, 2019 16:58
@dagwieers
Copy link
Collaborator Author

Ok, I fixed that as well.

@dagwieers dagwieers force-pushed the kidszone branch 5 times, most recently from d41352f to 124c1f1 Compare September 13, 2019 17:11
@dagwieers dagwieers marked this pull request as ready for review September 13, 2019 17:54
@dagwieers dagwieers force-pushed the kidszone branch 2 times, most recently from 25630a0 to 783ec93 Compare September 13, 2019 17:58
This makes the Kids zone part of the routing.
@dagwieers
Copy link
Collaborator Author

Voor mij is dit klaar. Als er een beslissing is over de settings, dan kunnen we dat achteraf aanpassen.

@michaelarnauts michaelarnauts merged commit c26890e into add-ons:master Sep 13, 2019
@michaelarnauts
Copy link
Collaborator

Thanks!

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.

None yet

2 participants