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

Introduce support for APL-A #21

Closed
wants to merge 4 commits into from
Closed

Introduce support for APL-A #21

wants to merge 4 commits into from

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 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.

    Support testing APLA directives #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
shouldEndSession has a special meaning when it is undefined. It is
usually used on APL devices to leave the visuals showing without opening
up the mic.

I modified most tests to explicity end the session.

The touch handler test was written to leave the shouldEndSession in an undefined
state to test the changes in this commit.

https://developer.amazon.com/en-US/docs/alexa/custom-skills/manage-skill-session-and-session-attributes.html#screen-session
Documentation changes after adding support for shouldEndSession being undefined.
@s-maheshbabu
Copy link
Contributor Author

Some unrelated commits got in. Will submit again.

@s-maheshbabu s-maheshbabu deleted the master branch December 19, 2020 04:51
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.

1 participant