Skip to content

✅Add validation rules for amp-autocomplete[inline] and amp-autocomplete[query]#25870

Merged
caroqliu merged 3 commits intoampproject:masterfrom
caroqliu:inline-val
Dec 5, 2019
Merged

✅Add validation rules for amp-autocomplete[inline] and amp-autocomplete[query]#25870
caroqliu merged 3 commits intoampproject:masterfrom
caroqliu:inline-val

Conversation

@caroqliu
Copy link
Copy Markdown
Contributor

@caroqliu caroqliu commented Dec 4, 2019

No description provided.

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Dec 4, 2019

Hey @ampproject/wg-caching, these files were changed:

  • extensions/amp-autocomplete/0.1/test/validator-amp-autocomplete.html
  • extensions/amp-autocomplete/0.1/test/validator-amp-autocomplete.out
  • extensions/amp-autocomplete/validator-amp-autocomplete.protoascii

@wassgha
Copy link
Copy Markdown
Contributor

wassgha commented Dec 5, 2019

Restarting percy

@caroqliu
Copy link
Copy Markdown
Contributor Author

caroqliu commented Dec 5, 2019

Added one more restriction: src should be used with query, ptal at the most recent commit when you get a chance. @Gregable @wassgha Thanks!

@wassgha wassgha self-requested a review December 5, 2019 19:42
@caroqliu caroqliu merged commit 5161a02 into ampproject:master Dec 5, 2019
@caroqliu caroqliu deleted the inline-val branch December 5, 2019 21:35
amaltas added a commit that referenced this pull request Dec 11, 2019
* cl/283618549 Revision bump for #25847

* cl/283654708 mandatory_parent can use spec_name in addition to tag_name

* cl/283852039 Introduce a invalid doctype error for amp validation.

* cl/283882898 n/a

* cl/283993894 Revision bump for #25197

* cl/284115876 Revision bump for #25870

* cl/284258503 Revision bump for #25889

* cl/284856390 Revision bump for #25946
micajuine-ho pushed a commit to micajuine-ho/amphtml that referenced this pull request Dec 27, 2019
…te[query] (ampproject#25870)

* Add validation rules for "inline", "query" attr

* Require "src" with "query"

* Alphabetize attributes
micajuine-ho pushed a commit to micajuine-ho/amphtml that referenced this pull request Dec 27, 2019
* cl/283618549 Revision bump for ampproject#25847

* cl/283654708 mandatory_parent can use spec_name in addition to tag_name

* cl/283852039 Introduce a invalid doctype error for amp validation.

* cl/283882898 n/a

* cl/283993894 Revision bump for ampproject#25197

* cl/284115876 Revision bump for ampproject#25870

* cl/284258503 Revision bump for ampproject#25889

* cl/284856390 Revision bump for ampproject#25946
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants