Skip to content

Show the shots of every episode in All shots playlists - #2281

Open
NicoPennec wants to merge 4 commits into
cgwire:mainfrom
NicoPennec:all-shots
Open

NicoPennec wants to merge 4 commits into
cgwire:mainfrom
NicoPennec:all-shots

Conversation

@NicoPennec

Copy link
Copy Markdown
Member

Problem

  • Under All shots, the playlists page lists only the shots already loaded, often none.
  • Shots of different episodes show the same sequence name there.
  • Add entire episode adds every shot of the production under All shots.

Solution

  • Load the shots of every episode under All shots.
  • Show the episode before the sequence, on the playlists page and in the playlist of a selection.
  • Hide the entire episode addition under All shots and the main pack.

Goes with cgwire/zou#1295

Under All shots, the playlist page loaded no shots and listed only the
ones the previous page left in the store: none after a reload, a single
episode after an episode switch. The entries of the other episodes were
dropped from the player as well.

It now loads the shots of the whole production, as the shots page does.
Sequence names repeat across episodes: under All shots, the addition
panel and the playlist entries showed several SQ01 with nothing to tell
them apart.

The sequence name now comes with its episode, as on the shots page.
…main pack

Under All shots, the loaded shots are those of the whole production:
Add entire episode added every shot of every episode to the playlist.
The main pack loads no shots, so there it added the shots another page
had left loaded.

Neither offers the entire episode or movie addition now. The pending
builds and the sequence additions stay.
The playlist generated from a selection of the shots page under All
shots showed SQ01 / SH010 for the SH010 of every episode.

Its entries now name the episode too, as on the playlist page.
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