Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make @frontity/smart-ads AMP-aware [8pt] #713

Open
SantosGuillamot opened this issue Feb 9, 2021 · 0 comments
Open

Make @frontity/smart-ads AMP-aware [8pt] #713

SantosGuillamot opened this issue Feb 9, 2021 · 0 comments
Labels
amp
Projects
AMP feature Sprint 11 Sprint 12 Sprint 13

Comments

@SantosGuillamot
Copy link
Member

This is needed for AMP feature. For full context please check out the final Implementation proposal.

The AMP framework uses a special syntax for the ads, so we have to make this package AMP-aware to load the correct content if the AMP package is active.
@SantosGuillamot SantosGuillamot added this to To do in AMP feature via automation Feb 9, 2021
@SantosGuillamot SantosGuillamot added this to To do in Sprint 11 via automation Feb 11, 2021
@SantosGuillamot SantosGuillamot moved this from To do to Backlog in Sprint 11 Feb 11, 2021
@SantosGuillamot SantosGuillamot added this to To do in Sprint 12 via automation Feb 25, 2021
@SantosGuillamot SantosGuillamot added this to To do in Sprint 13 via automation Mar 18, 2021
@SantosGuillamot SantosGuillamot moved this from To do to In progress in AMP feature Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
amp
Projects
AMP feature
  
In progress
Sprint 13
  
To do
Sprint 11
  
Backlog
Sprint 12
  
To do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SantosGuillamot