Skip to content

✨ amp-connatix-player: iframe domain based on a property#35179

Merged
kristoferbaxter merged 11 commits intoampproject:mainfrom
gab-kalmar:amp-connatix-player-domain-based-on-prop
Aug 3, 2021
Merged

✨ amp-connatix-player: iframe domain based on a property#35179
kristoferbaxter merged 11 commits intoampproject:mainfrom
gab-kalmar:amp-connatix-player-domain-based-on-prop

Conversation

@gaborkalmar
Copy link
Copy Markdown
Contributor

updated iframeDomain_ to be set based on a property ( data-elements-player )

@amp-owners-bot amp-owners-bot bot requested a review from nainar July 12, 2021 08:24
@amp-owners-bot
Copy link
Copy Markdown

Hey @alanorozco! These files were changed:

extensions/amp-connatix-player/0.1/amp-connatix-player.js
extensions/amp-connatix-player/0.1/test/test-amp-connatix-player.js

@alanorozco alanorozco removed the request for review from nainar July 13, 2021 15:37
@alanorozco
Copy link
Copy Markdown
Member

@kristoferbaxter kristoferbaxter merged commit 5f3a234 into ampproject:main Aug 3, 2021
westonruter added a commit to westonruter/amphtml that referenced this pull request Aug 3, 2021
…tok-validation

* 'main' of github.com:ampproject/amphtml: (72 commits)
  build: run amp lint --fix to address import order of jixie (ampproject#35513)
  ✨ [amp-analytics] Add Custom Browser Event Tracker (ampproject#35193)
  babel: teach amp mode transformer about #core/mode (ampproject#35477)
  🚮 Remove experiment `amp-consent-granular-consent` (ampproject#35508)
  ♻️ Enable auto-sorting+grouping within src/ and 3p/ (ampproject#35454)
  🐛  [amp-render] fix root-element stripping from amp-render with amp-bind (ampproject#35449)
  ✅ [Story interactive] Add Example Story for Detailed Results Component (ampproject#35450)
  🐛 Fix error on bento example (ampproject#35490)
  🐛 amp-story-grid-layer: Fix AMP invalidation error in documentation (ampproject#35503)
  🐛 Fix code formatting (ampproject#35499)
  ✅ buildDom: add tests for amp-fit-text and amp-layout (ampproject#35494)
  ♻️ Remove unused imports (ampproject#35435)
  ✨ amp-connatix-player: iframe domain based on a property (ampproject#35179)
  Updated document with use cases of remote config (ampproject#35496)
  AMP.goBack: update documentation (ampproject#29290)
  🏗 Allow the bundle-size job to run even if the builds were skipped (ampproject#35492)
  build-system: improve terser/esbuild integration (ampproject#35466)
  🧪 [Story performance] Disable animations on first page to 50% (ampproject#35476)
  📖 [Amp story] [Page attachments] Amp.dev Docs for New Page Attachment Features ampproject#34883 (ampproject#35338)
  🚀 [Story interactive] Rewrite Image URL to Cached URL for Image Quizzes and Polls (ampproject#35375)
  ...
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.

4 participants