Skip to content

Added documentation for A4A#7233

Merged
cramforce merged 10 commits intoampproject:masterfrom
google:frizz-add-a4a-docs
Jan 27, 2017
Merged

Added documentation for A4A#7233
cramforce merged 10 commits intoampproject:masterfrom
google:frizz-add-a4a-docs

Conversation

@bradfrizzell
Copy link
Copy Markdown
Contributor

Added the Network Implementation Guide
Added the A4A Readme

@jasti
Copy link
Copy Markdown
Contributor

jasti commented Jan 27, 2017

LGTM. @lannka, can you please merge ASAP. Thanks.

@dknecht
Copy link
Copy Markdown
Contributor

dknecht commented Jan 27, 2017

Can we add diagram that show inline add signing through a proxy? Or do you prefer follow up pull?

@dknecht
Copy link
Copy Markdown
Contributor

dknecht commented Jan 27, 2017

cc: @ampproject/cloudflare

@bradfrizzell
Copy link
Copy Markdown
Contributor Author

Up to Vamsee, I know it's very important this be merged today.

@jasti
Copy link
Copy Markdown
Contributor

jasti commented Jan 27, 2017

Lets merge for now, but @dknecht feel free to send a follow up PR.

@bradfrizzell
Copy link
Copy Markdown
Contributor Author

hongfei is on vacation as I understand, @erwinmombay can you please merge ASAP? thanks :D

@@ -0,0 +1,48 @@
# A4A - AMP for Ads
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've seen marketing slides calling them AMP Ads now

@cramforce cramforce merged commit 78dd260 into ampproject:master Jan 27, 2017
jridgewell pushed a commit to jridgewell/amphtml that referenced this pull request Jan 31, 2017
torch2424 pushed a commit to torch2424/amphtml that referenced this pull request Feb 14, 2017
mrjoro pushed a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants