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.

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

🐛✅TE Medya Ad Type #26757

Merged
merged 18 commits into from
Feb 12, 2020
Merged

🐛✅TE Medya Ad Type #26757

merged 18 commits into from
Feb 12, 2020

Conversation

vidyome
Copy link
Contributor

@vidyome vidyome commented Feb 12, 2020

No description provided.

@vidyome
Copy link
Contributor Author

vidyome commented Feb 12, 2020

We expect it to be merged. @powerivq
Best regards. 👍 🥇

@powerivq powerivq merged commit 728a0d0 into ampproject:master Feb 12, 2020
vidyome added a commit to vidyome/amphtml that referenced this pull request Feb 12, 2020
@vidyome
Copy link
Contributor Author

vidyome commented Feb 12, 2020

Hello @powerivq , I pressed the revert button unintentionally, do I need to take any action?
When will the amp-ads js library be active?
Best regards.

@powerivq
Copy link
Contributor

@vidyome No consequences for that. It'll be on canary next Tuesday and prod the next Tuesday after that.

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

Successfully merging this pull request may close these issues.

4 participants