prebid / prebid-mobile-android

Prebid Mobile SDK for Android applications
Apache License 2.0
58 stars 84 forks source link

Additional attribute VideoParameters to indicates video to be skipped #756

Open andriiginting opened 5 months ago

andriiginting commented 5 months ago

Is your feature request related to a problem? Please describe. as stated in OpenRTB API Spec v2.5, we require to send a value to indicate whether the video object is skippable or not

Describe the solution you'd like Based on OpenRTB API specification doc section Object: Video, one of the attributes is skip which indicates if the player allows the video to be skipped Add a new attribute in VideoParameters to control skippable video objects with an integer value

/* 
* VideoParameters.java
**/

@Nullable
    public Integer getSkippable() {
        return skippable;
    }

    public void setSkippable(@Nullable Integer skippable) {
        this.skippable = skippable;
    }

Describe alternatives you've considered

Additional context I'm working in a news company and our partner asked to send this value through Prebid SDK

jsligh commented 5 months ago

@andriiginting skip support is currently being discussed and proposed for version 3.0.

andriiginting commented 5 months ago

Hi @jsligh, would you mind sharing the timeline for the 3.0.0 version? Is it going to implement both Android and iOS platforms?

jsligh commented 5 months ago

@andriiginting here is the plan for 3.0.0: https://docs.prebid.org/prebid-mobile/docs/Prebid_Mobile_3_0_0.pdf

There is no timeline yet but we are looking to get the plan finalized.

andriiginting commented 5 months ago

is there any chance that the next version will include battr as well? @jsligh ref: 6.3.4 Example 4 - Video

andriiginting commented 5 months ago

Hi @jsligh @YuriyVelichkoPI, want to follow up regarding the timeline. do you have an estimation when the timeline for 3.0.0 is ready?

jsligh commented 5 months ago

@andriiginting we are looking to have 3.0.0 complete by Q4 of this year. Can you open another issue about including battr?