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
✅ Add validation rules for amp-autocomplete. #21526
✅ Add validation rules for amp-autocomplete. #21526
Conversation
@honeybadgerdontcare (or @Gregable ): I discussed this with @caroqliu and we were both puzzling over how to accomplish the above validation logic. Any advice? |
@cvializ yes, this is something I'm looking into |
Thanks! |
extensions/amp-autocomplete/validator-amp-autocomplete.protoascii
Outdated
Show resolved
Hide resolved
1591ae0
to
604556f
Compare
Made the changes we discussed offline yesterday, ptal @cvializ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
validation looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
* Create validation files * Add validation test HTML file * Add validator rules and update out * Update validator rules * Attempt validate template usage * Undo nested input * Modify validation rules * Change "value" to "data-value" * Remove unwanted valid cases from validator tests * Update data-value in test * minor nits
* cl/243147325 Revision bump for ampproject#21770 * cl/244691564 Revision bump for ampproject#21552 * cl/245095560 Allow i-amphtml-sizer as child of amp-image-slider. * cl/245331243 Allow i-amphtml-sizer as child of amp-video-iframe. * cl/245480181 Revision bump for ampproject#21526
No description provided.