Skip to content

feat: Добавлен сахар: дополнительные методы подписки на команды сценария #256

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

evgeniysemin
Copy link
Contributor

@evgeniysemin evgeniysemin commented Jan 25, 2022

📦 Published PR as canary version: 4.17.0--canary.256.004c305de8dd0429be86cdeb3b7d18e7a7e5457e.0

✨ Test out this PR locally via:

npm install @sberdevices/assistant-client@4.17.0--canary.256.004c305de8dd0429be86cdeb3b7d18e7a7e5457e.0
# or 
yarn add @sberdevices/assistant-client@4.17.0--canary.256.004c305de8dd0429be86cdeb3b7d18e7a7e5457e.0

@evgeniysemin evgeniysemin linked an issue Jan 25, 2022 that may be closed by this pull request
src/createAssistant.ts Outdated Show resolved Hide resolved
src/createAssistant.ts Outdated Show resolved Hide resolved
@evgeniysemin evgeniysemin changed the title feat: Добавлены специальные методы подписки на smart_app_error и smart_app_data feat: Добавлен сахар: дополнительные методы подписки на комманды сценария Feb 16, 2022
@evgeniysemin evgeniysemin changed the title feat: Добавлен сахар: дополнительные методы подписки на комманды сценария feat: Добавлен сахар: дополнительные методы подписки на команды сценария Feb 16, 2022
@evgeniysemin evgeniysemin force-pushed the issues/247-subscribeToCommand branch 2 times, most recently from 004638d to 8374a0c Compare February 16, 2022 11:17
}
});
},
onSmartAppError: (subscriber: (smartAppError: AssistantSmartAppError['smart_app_error']) => void) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

нужно убрать, дублирует on('error')

}
});
},
onAssistantCommand: (
Copy link
Contributor

Choose a reason for hiding this comment

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

попробовать сделать один метод для подписки на команды и smart_app_data

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Расширить api новыми методами
2 participants