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

✨ Lotame vendor - set disableKeyAppend to true #18051

Merged
merged 1 commit into from Sep 18, 2018

Conversation

Tonsil
Copy link
Contributor

@Tonsil Tonsil commented Sep 14, 2018

This change is in response to complaints from our clients. When we changed from the old remote.html implementation to RTC, the suffix "_lotame" started to be added to variable names. This was causing confusion, and it was decided that everyone would be happier if we simply disabled that behavior.

@bradfrizzell bradfrizzell merged commit 7f5fa60 into ampproject:master Sep 18, 2018
@Tonsil Tonsil deleted the lotame_disable_key_append branch September 19, 2018 14:10
torch2424 pushed a commit to torch2424/amphtml that referenced this pull request Sep 20, 2018
torch2424 pushed a commit to torch2424/amphtml that referenced this pull request Oct 10, 2018
Enriqe pushed a commit to Enriqe/amphtml that referenced this pull request Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants