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

Change /c/s/ to /ad/s/ for template ads. #13175

Merged
merged 3 commits into from Jan 31, 2018
Merged

Change /c/s/ to /ad/s/ for template ads. #13175

merged 3 commits into from Jan 31, 2018

Conversation

lannka
Copy link
Contributor

@lannka lannka commented Jan 31, 2018 •
edited

For #12903

@lannka lannka requested a review from glevitzky January 31, 2018 16:04
@lannka lannka merged commit 757d52e into ampproject:master Jan 31, 2018
@lannka lannka deleted the ad branch January 31, 2018 17:56
RanAbram pushed a commit to RanAbram/amphtml that referenced this pull request Mar 12, 2018
@lannka @RanAbram
Change /c/s/ to /ad/s/ for template ads. (ampproject#13175)
cd2baea 
* Change /c/s/ to /ad/s/ for template ads.

* Change /c/s/ to /ad/s/ for template ads.

* Fix test
protonate pushed a commit to protonate/amphtml that referenced this pull request Mar 15, 2018
@lannka
Change /c/s/ to /ad/s/ for template ads. (ampproject#13175)
faf4cf2 
* Change /c/s/ to /ad/s/ for template ads.

* Change /c/s/ to /ad/s/ for template ads.

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

@glevitzky glevitzky glevitzky approved these changes

Assignees
No one assigned
Labels
cla: yes PR use: In Beta / Experimental PR use: In Stable
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lannka @glevitzky @googlebot @ampprojectbot