-
Notifications
You must be signed in to change notification settings - Fork 70
Open
Description
Throughout seemingly all of this library, the Amazon App ID is defined in either the variable or a property called "applicationId". I noticed some areas where it was called "applicationid" instead, as well a few instances where it was missing as an argument:
IntentRequest.php
Line 9/10 - Missing $applicationId as 2nd argument in __construct function.
Request.php
Line 94 - Missing $this->applicationId as 2nd argument in new Request class creation.
These were reported in my Drupal 8 error logs while I build an Alexa integration app. I am new to this library though, so if I am incorrectly reporting this as an issue for some reason, please let me know so I can correct this.
Metadata
Metadata
Assignees
Labels
No labels