alexa / alexa-skills-kit-sdk-for-nodejs

The Alexa Skills Kit SDK for Node.js helps you get a skill up and running quickly, letting you focus on skill logic instead of boilerplate code.
Apache License 2.0
3.13k stars 738 forks source link

ReminderManagerServiceClient.deleteReminders() is not a function #548

Closed TommyBs closed 5 years ago

TommyBs commented 5 years ago

I'm submitting a...


[ ] Regression (a behavior that used to work and stopped working in a new release)
[X ] Bug report  
[ ] Performance issue
[ ] Feature request
[X ] Documentation issue or request
[ ] Other... Please describe:

Expected Behavior

Calling

const ReminderManagementServiceClient = serviceClientFactory.getReminderManagementServiceClient();

       await ReminderManagementServiceClient.deleteReminders();

should delete all reminders. But it actually throws an error that deleteReminders() is not a function yet it is listed in the docs.

It should delete all reminders.

Current Behavior

An error is thrown saying deleteReminders is not a function.

Error handled: ReminderManagementServiceClient.deleteReminders is not a function TypeError: ReminderManagementServiceClient.deleteReminders is not a function at Object.handle (/var/task/index.js:225:63) at GenericHandlerAdapter. (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:63:47) at step (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:44:23) at Object.next (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:25:53) at /var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:19:71 at new Promise () at __awaiter (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:15:12) at GenericHandlerAdapter.execute (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/request/handler/GenericHandlerAdapter.js:61:16) at GenericRequestDispatcher. (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/GenericRequestDispatcher.js:173:58) at step (/var/task/node_modules/ask-sdk-runtime/dist/dispatcher/GenericRequestDispatcher.js:44:23)

Possible Solution

Either delete all the reminders or update the docs to highlight this isn't a function

Steps to Reproduce (for bugs)

Create some reminders with a skill, then try to delete them all with the service client. e.g.

Context

Docs -> https://ask-sdk-for-nodejs.readthedocs.io/en/latest/Calling-Alexa-Service-APIs.html?highlight=permissions#remindermanagementserviceclient

Your Environment

nikhilym commented 5 years ago

Hey @TommyBs , thanks for getting this to our attention. This is a documentation issue in our SDK docs. You can check on the Reminder Management API Reference developer doc that there is no such API delete Reminders available for use. We will update our docs to get this removed.