SRGSSR / pillarbox-documentation

Technical cross-platform documentation for Pillarbox
https://srgssr.github.io/pillarbox-documentation
MIT License
3 stars 0 forks source link

Discuss nested and overlapping blocked time range support #68

Closed defagos closed 2 months ago

defagos commented 5 months ago

As an editor I want blocked time ranges not to be reachable, even if nested or overlapping. As a Pillarbox development team we would like to possibly have a common answer to this need.

Acceptance criteria

Hints

Tasks

defagos commented 4 months ago

Discussed within the team.

Current frontend status

Desired frontend status and next steps

We would like to eliminate all too clever code from clients or to avoid introducing clever code by defining overlaps or nested ranges as undefined behavior. Time range overlap or nesting should be addressed once server-side.

To know if this ideal is possible we must first:

  1. Before any code change ask BUs if overlapping / nested segments are still a thing and if in such cases being able to briefly see a forbidden frame is a problem.
  2. If yes discuss the matter with the backend team so that segments are flattened server-side. Explain that clients otherwise will exhibit undefined behavior. Create a dedicated story if the backend team agree.
  3. If the backend team refuse and the feature is essential, keep it in the Apple code.

In all cases no additional work is required for the web and Android. On all platforms examples with disjoint time ranges should available. If (and only if) overlapping and nesting is important from a business perspective should the demos contain dedicated examples, otherwise any existing example must be removed.

defagos commented 4 months ago

The discussion has been started with the backend team but is not finished yet.

defagos commented 3 months ago

@amtins will slightly revisit the ticket linked on Confluence as its title / content might be slightly misleading.

defagos commented 3 months ago

A new issue has been created in the backend backlog.

defagos commented 2 months ago

We have decided to keep the topic for future SAM services, thus avoiding work on the legacy IL services. The associated JIRA issue has been closed.