search

pusher / push-notifications-web

Beams Browser notifications
MIT License
39 stars 19 forks source link

`getDeviceInterests` doesn't support pagination as per API docs #115

Closed Harsh-br0 closed 1 year ago

Harsh-br0 commented 1 year ago

This is a follow-up of #80 As per this issue-comment

Bug Report

The getDeviceInterests method does not implement the pagination protocol used by the handler

Reproduction Steps

Call getDeviceInterests on a device that has > 100 subscriptions

Expected Result

All device interests should be returned

Actual Result

Only the first 100 interests (the first page) will be returned

britneycodes commented 1 year ago

Hey @Harsh-br0 , thanks for your submission! Could I have your email address so we can send you some swag?

Harsh-br0 commented 1 year ago

Could I have your email address so we can send you some swag?

Sure @britneycodes , Here it is ThatsHarsh@proton.me

Edit:

I haven't received any email yet...