New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability for amp-ad to refresh itself after declared time #4038

Closed
jasti opened this Issue Jul 12, 2016 · 17 comments

Comments

Projects
None yet
@jasti
Collaborator

jasti commented Jul 12, 2016

At the moment amp-ad doesn't have a way to refresh itself. Some ad servers would like the ability to refresh themselves after the user has stayed on the page for a certain time period.
@cramforce

@lannka

This comment has been minimized.

Show comment
Hide comment
@lannka

lannka Jul 13, 2016

Collaborator

I suppose this is speaking about a4a?

For the current amp-ad, it's up to the ad network's SDK to do periodical refresh.

Collaborator

lannka commented Jul 13, 2016

I suppose this is speaking about a4a?

For the current amp-ad, it's up to the ad network's SDK to do periodical refresh.

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Jul 13, 2016

Collaborator

No, don't mean A4A.
At an amp-ad level, we should allow publishers to set how frequently they'd like an ad to refresh. So the runtime can take care of invoking this across any ad server. Any downsides?

Collaborator

jasti commented Jul 13, 2016

No, don't mean A4A.
At an amp-ad level, we should allow publishers to set how frequently they'd like an ad to refresh. So the runtime can take care of invoking this across any ad server. Any downsides?

@cramforce

This comment has been minimized.

Show comment
Hide comment
@cramforce

cramforce Jul 13, 2016

Member

I think it makes sense to implement on the AMP level as this is something that could be implemented as a cross cutting thing, independent of the ad network.

It be interesting to know how many publishers would use it.

Member

cramforce commented Jul 13, 2016

I think it makes sense to implement on the AMP level as this is something that could be implemented as a cross cutting thing, independent of the ad network.

It be interesting to know how many publishers would use it.

@ericlindley-g

This comment has been minimized.

Show comment
Hide comment
@ericlindley-g

ericlindley-g Jul 13, 2016

Collaborator

Slotting into the backlog for now, but please correct/update as needed

Collaborator

ericlindley-g commented Jul 13, 2016

Slotting into the backlog for now, but please correct/update as needed

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Jul 19, 2016

Collaborator

@ampproject/ads Adserver folks, looking for some input here. Do you hear this feature request from publishers you support? Do publishers want to auto-refresh a slot on a page? if so, priority?

Collaborator

jasti commented Jul 19, 2016

@ampproject/ads Adserver folks, looking for some input here. Do you hear this feature request from publishers you support? Do publishers want to auto-refresh a slot on a page? if so, priority?

@Rooke

This comment has been minimized.

Show comment
Hide comment
@Rooke

Rooke Jul 19, 2016

Contributor

The demand for this among publishers exists. Sortable had enough of these requests that we allowed the data-ad-refresh="user 30s event 30s time 30s" property to give publishers the ability to specify how they wanted to refresh ads. e.g. If the data-ad-refresh is set to a time (say 30s), the unit will automatically refresh after 30s in the viewport. I wouldn't call this a high priority issue though, not that many pubs use this feature.

Contributor

Rooke commented Jul 19, 2016

The demand for this among publishers exists. Sortable had enough of these requests that we allowed the data-ad-refresh="user 30s event 30s time 30s" property to give publishers the ability to specify how they wanted to refresh ads. e.g. If the data-ad-refresh is set to a time (say 30s), the unit will automatically refresh after 30s in the viewport. I wouldn't call this a high priority issue though, not that many pubs use this feature.

@yieldmo-rao

This comment has been minimized.

Show comment
Hide comment
@yieldmo-rao

yieldmo-rao Jul 19, 2016

Contributor

Hello all,

From our end, we actually haven't had any requests for auto ad refreshes on
AMP pages. Just as an FYI, we have a couple publishers on regular mobile
web (that are on DFP) that use the DFP auto refresh feature, but even those
pubs are few, and haven't requested the same on AMP.

Best,

Rahul

On Tue, Jul 19, 2016 at 4:43 PM, Michael Rooke notifications@github.com
wrote:

The demand for this among publishers exists. Sortable had enough of these
requests that we allowed the data-ad-refresh="user 30s event 30s time 30s"
property to give publishers the ability to specify how they wanted to
refresh ads. e.g. If the data-ad-refresh is set to a time (say 30s), the
unit will automatically refresh after 30s in the viewport. I wouldn't call
this a high priority issue though, not that many pubs use this feature.


You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
#4038 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOZbzx7gBcy57F0RpKvIXEYaYWjRZKquks5qXTb1gaJpZM4JK73u
.

Contributor

yieldmo-rao commented Jul 19, 2016

Hello all,

From our end, we actually haven't had any requests for auto ad refreshes on
AMP pages. Just as an FYI, we have a couple publishers on regular mobile
web (that are on DFP) that use the DFP auto refresh feature, but even those
pubs are few, and haven't requested the same on AMP.

Best,

Rahul

On Tue, Jul 19, 2016 at 4:43 PM, Michael Rooke notifications@github.com
wrote:

The demand for this among publishers exists. Sortable had enough of these
requests that we allowed the data-ad-refresh="user 30s event 30s time 30s"
property to give publishers the ability to specify how they wanted to
refresh ads. e.g. If the data-ad-refresh is set to a time (say 30s), the
unit will automatically refresh after 30s in the viewport. I wouldn't call
this a high priority issue though, not that many pubs use this feature.


You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
#4038 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOZbzx7gBcy57F0RpKvIXEYaYWjRZKquks5qXTb1gaJpZM4JK73u
.

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Jul 20, 2016

Collaborator

Thanks @Rooke and @yieldmo-rao! We'll slot this into backlog unless any other adserver objects and work on higher priority things.

Collaborator

jasti commented Jul 20, 2016

Thanks @Rooke and @yieldmo-rao! We'll slot this into backlog unless any other adserver objects and work on higher priority things.

@jmheist

This comment has been minimized.

Show comment
Hide comment
@jmheist

jmheist Dec 6, 2016

@jasti, Meredith is looking for this feature, or a way to hack it ourselves. any chance this will move out of the backlog soon?

jmheist commented Dec 6, 2016

@jasti, Meredith is looking for this feature, or a way to hack it ourselves. any chance this will move out of the backlog soon?

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Dec 6, 2016

Collaborator

Hey @jmheist - I don't think this is being looked at as priority. We'd welcome if you are willing to implement the ability to refresh ads as a general functionality in amp-ad.
Would you be interested? If so, could you please create an ITI?
Thanks!

Collaborator

jasti commented Dec 6, 2016

Hey @jmheist - I don't think this is being looked at as priority. We'd welcome if you are willing to implement the ability to refresh ads as a general functionality in amp-ad.
Would you be interested? If so, could you please create an ITI?
Thanks!

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Feb 7, 2017

Collaborator

Closing because of lack of priority.

Collaborator

jasti commented Feb 7, 2017

Closing because of lack of priority.

@jasti jasti closed this Feb 7, 2017

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Mar 22, 2017

Collaborator

Reopening because of request from the publisher working group.

Collaborator

jasti commented Mar 22, 2017

Reopening because of request from the publisher working group.

@jasti jasti reopened this Mar 22, 2017

@jasti jasti modified the milestones: Prioritized FRs, Pending Triage Mar 22, 2017

@jmheist

This comment has been minimized.

Show comment
Hide comment
@jmheist

jmheist Mar 22, 2017

jmheist commented Mar 22, 2017

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Mar 22, 2017

Collaborator

Please join here: https://groups.google.com/forum/#!forum/amp-news-publishers-wg
We had a session this morning and slides from the session are posted to the group.

Collaborator

jasti commented Mar 22, 2017

Please join here: https://groups.google.com/forum/#!forum/amp-news-publishers-wg
We had a session this morning and slides from the session are posted to the group.

@jasti

This comment has been minimized.

Show comment
Hide comment
@jasti

jasti Mar 30, 2017

Collaborator

This can be split into 3 sub features:

  • Ability to refresh a sticky ad with a specified time limit. Minimum is 30 seconds.
  • Ability to refresh any ad with a specified time limit. Minimum is 30 seconds.
  • Ability to refresh ad ad when a user scroll back or forward an that was already previously viewed as if the ad is being loaded for the first time.
Collaborator

jasti commented Mar 30, 2017

This can be split into 3 sub features:

  • Ability to refresh a sticky ad with a specified time limit. Minimum is 30 seconds.
  • Ability to refresh any ad with a specified time limit. Minimum is 30 seconds.
  • Ability to refresh ad ad when a user scroll back or forward an that was already previously viewed as if the ad is being loaded for the first time.
@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Apr 17, 2017

We have a client that just asked for this as well.

ghost commented Apr 17, 2017

We have a client that just asked for this as well.

@keithwrightbos

This comment has been minimized.

Show comment
Hide comment
@keithwrightbos

keithwrightbos Apr 18, 2017

Collaborator
Collaborator

keithwrightbos commented Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment