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

Apla #22

Merged
merged 2 commits into from
Dec 13, 2020
Merged

Apla #22

merged 2 commits into from
Dec 13, 2020

Conversation

s-maheshbabu
Copy link
Contributor

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Introduces support for APL-A

APL-A is Alexa Presentation Language for audio effects. It is similar to
APL in its structure (directives, data sources, documents etc.). However,
it is different from APL in that there is no concept of supported interfaces.
It is supported on all Alexa devices.

In this commit, we add support for validating APLA directives.

I was divided on whether we should just add APL-A to existing APL validator
or if we should create a completely independent validator for APL-A. For now
I added it to the existing APLValidator but open to feedback.

#20

APL-A is Alexa Presentation Language for audio effects. It is similar to
APL in its structure (directives, data sources, documents etc.). However,
it is different from APL in that there is no concept of supported interfaces.
It is supported on all Alexa devices.

In this commit, we add support for validating APLA directives and also added
a test.

I was divided on whether we should just add APL-A to existing APL validator
or if we should create a compeltely independent validator for APL-A. For now
I added it to the existing APLValidator but open to feedback.

#20
@hoegertn hoegertn merged commit 6129966 into taimos:master Dec 13, 2020
@s-maheshbabu s-maheshbabu deleted the apla branch December 13, 2020 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants