I propose the following enhancement to the playlist generation that happens during startup. Originally I was looking for just the ability to play only the night video's, but after looking at the JSON I see there is at least one extra data point that can be used.
Playlist Options
Order:
:radio_button: Shuffle/Random
:white_circle: Original
Time of Day:
:radio_button: No Preference
:white_circle: Time of day (play all night or all day videos first)
:white_circle: Only day videos
:white_circle: Only night videos
Locations:
:radio_button: All
:white_circle: Specific
:white_square_button: China
:white_square_button: Hawaii
:white_square_button: London
:white_square_button: New York City
:white_square_button: San Francisco
I think if the logic to pick the next video is moved out of the playlist generation and into the SetNextVideo (or refactored into a class that handles both). This would be cleaner and would make it possible to support a more user friendly option of "Play video's matching the current time of day", rather than generating the playlist once during startup and simply ordering the array and iterating through it.
I propose the following enhancement to the playlist generation that happens during startup. Originally I was looking for just the ability to play only the night video's, but after looking at the JSON I see there is at least one extra data point that can be used.
Playlist Options
Order: :radio_button: Shuffle/Random :white_circle: Original
Time of Day: :radio_button: No Preference :white_circle: Time of day (play all night or all day videos first) :white_circle: Only day videos :white_circle: Only night videos
Locations: :radio_button: All :white_circle: Specific :white_square_button: China :white_square_button: Hawaii :white_square_button: London :white_square_button: New York City :white_square_button: San Francisco
I think if the logic to pick the next video is moved out of the playlist generation and into the SetNextVideo (or refactored into a class that handles both). This would be cleaner and would make it possible to support a more user friendly option of "Play video's matching the current time of day", rather than generating the playlist once during startup and simply ordering the array and iterating through it.