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

fix: issue of parsing response for here_now #427

Merged
merged 5 commits into from
Jan 2, 2025

Conversation

mohitpubnub
Copy link
Contributor

@mohitpubnub mohitpubnub commented Jan 2, 2025

fix: fix hereNow response parsing for total occupancy count field.

Fixed issue of hereNow response parsing for totalOccupancy field.

Comment on lines 31 to 42
// describe('#whereNow', () => {
// it('returns the requested data for user UUID', (done) => {
// const scope = utils
// .createNock()
// .get('/v2/presence/sub-key/mySubscribeKey/uuid/myUUID')
// .query({
// pnsdk: `PubNub-JS-Nodejs/${pubnub.getVersion()}`,
// uuid: 'myUUID',
// })
// .reply(200, '{"status": 200, "message": "OK", "payload": {"channels": ["a","b"]}, "service": "Presence"}', {
// 'content-type': 'text/javascript',
// });
Copy link
Contributor

Choose a reason for hiding this comment

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

Woah, what happened here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my bad! Reverting in a second

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ignore for now

@mohitpubnub mohitpubnub self-assigned this Jan 2, 2025
@mohitpubnub mohitpubnub added type: bug This issue reports a bug. type: fix This PR contains fixes to existing features. labels Jan 2, 2025
@mohitpubnub
Copy link
Contributor Author

@parfeon Ready now for review

Copy link
Contributor

@parfeon parfeon left a comment

Choose a reason for hiding this comment

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

LGTM!

@mohitpubnub
Copy link
Contributor Author

@pubnub-release-bot release

@mohitpubnub mohitpubnub merged commit 1a2ddeb into master Jan 2, 2025
7 checks passed
@mohitpubnub mohitpubnub deleted the fix/hereNow_Occupancy branch January 2, 2025 13:02
@pubnub-release-bot
Copy link
Contributor

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug This issue reports a bug. type: fix This PR contains fixes to existing features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants