Skip to content

Allow AMP Stories to work in Reader mode#2143

Merged
westonruter merged 2 commits intoamp-stories-reduxfrom
stamp/enable-stories-in-reader-mode
Apr 17, 2019
Merged

Allow AMP Stories to work in Reader mode#2143
westonruter merged 2 commits intoamp-stories-reduxfrom
stamp/enable-stories-in-reader-mode

Conversation

@westonruter
Copy link
Copy Markdown
Member

Fixes #2137.

@westonruter westonruter requested a review from felixarntz April 17, 2019 08:36
@googlebot googlebot added the cla: yes Signed the Google CLA label Apr 17, 2019
Copy link
Copy Markdown
Collaborator

@felixarntz felixarntz left a comment

Choose a reason for hiding this comment

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

This is awesome!

@amedina
Copy link
Copy Markdown
Member

amedina commented Apr 17, 2019

Yes! This is super necessary.

@westonruter
Copy link
Copy Markdown
Member Author

Note: One side effect of the changes here (specifically 4645a4a) is that making changes to non-story posts when in Reader mode will start to show the validation error messages. Previously the validation warnings would only be shown when in Native/Transitional mode. The validation notices will likely be a source of complaints for users when in Reader mode, since they would be reported as if the Transitional mode is active.

@westonruter westonruter merged commit eb93ffc into amp-stories-redux Apr 17, 2019
@swissspidy swissspidy deleted the stamp/enable-stories-in-reader-mode branch May 14, 2019 18:07
@westonruter westonruter added this to the v1.2 milestone May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Signed the Google CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants