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

[Bug]: methods api.beginLiveActivity and api.endLiveActivity are missing, but documented #92

Open
1 task done
pepakriz opened this issue Dec 6, 2024 · 2 comments
Open
1 task done
Labels
bug Something isn't working

Comments

@pepakriz
Copy link

pepakriz commented Dec 6, 2024

What happened?

Based on readme.md these methods should be ready to use, but they are missing in the implementation.

Steps to reproduce?

yarn add @onesignal/node-onesignal

then

try to write the same code as in the readme:


...

// missing method
await api.beginLiveActivity('<app_id>', activityId, beginLiveActivityRequest);
// missing method
await api.endLiveActivity('<app_id>', '<activity_id>', subscriptionId);

What did you expect to happen?

I expect these methods to be available.

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@pepakriz pepakriz added the bug Something isn't working label Dec 6, 2024
@sherwinski
Copy link
Contributor

Hi @pepakriz,

Thank you for bringing this to our attention. I need to investigate a bit further but so far this is what I've gathered:

  • The beginLiveActivity and endLiveActivity endpoints were meant to be used in SDKs that ran on the users device, e.g. https://github.com/OneSignal/OneSignal-iOS-SDK, and as such did not make sense to support in this project
  • We are still missing an analog to start live activity, which will need to be added to this project. I've made a note to add that in with the next batch of updates that we release.

In the meantime, please let me know if you have any questions.

@pepakriz
Copy link
Author

Hi @sherwinski .

This endpoint https://documentation.onesignal.com/reference/start-live-activity is exactly what I'm missing in the SDK. It'd be great to have it in the next SDK release.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants