Move startup props into a single configuration

[armands-malejevs]

Feature Request

Create a new prop where all current props that are only used during initialisation and cannot be changed during runtime into a single configuration prop.

Why it is needed

This will make it more clear to developer as to which props can be changed during runtime and which props can only be provided when creating the player. This will also simplify the initialisation logic to get rid of postDelayed calls etc.

Tasks

• textTracks (external subtitle tracks)
• contentStartTime
• drm
• Ads props

[wood1986]

@freeboub and @armadilio3. Could you guys please come a list of props which is only for the player initialization? Any change to these props will be ignored after the initialization

fullscreen
controls

[wood1986]

I prefer breaking the people build in alpha after fixing this. I mean throw an exception when people still using original fullscreen and controls. Do you guys have any concerns?

[freeboub]

@wood1986 I already flag following props as 'startProps':

• textTracks (external subtitle tracks)
• bufferConfig
• contentStartTime
• drm
• header
• useTunnelingPlayback (a change from my working branch I need to backport)

The good news is that we should be able to implement this change without changing external API.
in Video.js, we can get public input paramters and reformat then to provide it to native.
That's my plan, but I have a lack of time :)

[freeboub]

• localSourceEncryptionKeyScheme for ios shall be moved to source.drm.localSourceEncryptionKeyScheme

[freeboub]

• localSourceEncryptionKeyScheme for ios shall be moved to source.drm.localSourceEncryptionKeyScheme

Done in: #4136 thank you @KrzysztofMoch

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[freeboub]

reviewing the description of the ticket, and it seems that headers property implementation is missing: https://thewidlarzgroup.github.io/react-native-video/component/props#headers
I cannot find implementation :/

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.

[github-actions]

Thank you for your feature request. We will review it and get back to you if we need more information.