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

846 Corrected schema definition for appD interop.intents.listensFor element #847

Merged
merged 2 commits into from
Nov 10, 2022

Conversation

kriswest
Copy link
Contributor

@kriswest kriswest commented Nov 3, 2022

resolves #846
appD schema contained an error that turned the interop.intents.listensFor element values (keyed on intent name) into an array of intents instead of a single intent definition - hence it did not match the example (which is correct according to the issue that introduced this element).

@kriswest kriswest added docs Documentation app-directory labels Nov 3, 2022
@kriswest kriswest requested review from robmoffat, hughtroeger and a team November 3, 2022 12:08
@netlify
Copy link

netlify bot commented Nov 3, 2022

Deploy Preview for lambent-kulfi-cf51a7 ready!

Name Link
🔨 Latest commit 796083c
🔍 Latest deploy log https://app.netlify.com/sites/lambent-kulfi-cf51a7/deploys/636cd300c29fff000863dc89
😎 Deploy Preview https://deploy-preview-847--lambent-kulfi-cf51a7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Member

@robmoffat robmoffat left a comment

Choose a reason for hiding this comment

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

I tested this out locally, lgtm

@kriswest kriswest merged commit 45e72d6 into master Nov 10, 2022
@kriswest kriswest deleted the 846-correct-interop.listensFor-element-definition branch November 10, 2022 10:31
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.

Example and Definition Don't Match For App Directory listensFor
4 participants