Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

clen1290/add optional param in setState for setting state through heartbeat endpoint#324

Merged
techwritermat merged 8 commits intomasterpubnub/javascript:masterfrom
CLEN-1290pubnub/javascript:CLEN-1290Copy head branch name to clipboard
Jun 14, 2023
Merged

clen1290/add optional param in setState for setting state through heartbeat endpoint#324
techwritermat merged 8 commits intomasterpubnub/javascript:masterfrom
CLEN-1290pubnub/javascript:CLEN-1290Copy head branch name to clipboard

Conversation

@mohitpubnub
Copy link
Contributor

feat: added support for setting state using heartbeat endpoint.

Added optional param withHeartbeat to set state through heartbeat endpoint.

@mohitpubnub mohitpubnub requested a review from parfeon as a code owner June 12, 2023 02:08
@mohitpubnub mohitpubnub changed the title add optional param in setState for setting state through heartbeat endpoint clen1290/add optional param in setState for setting state through heartbeat endpoint Jun 12, 2023
@mohitpubnub mohitpubnub self-assigned this Jun 12, 2023
@mohitpubnub mohitpubnub added the status: done This issue is considered resolved. label Jun 12, 2023
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 as v7.2.3

@techwritermat techwritermat merged commit 10df45b into master Jun 14, 2023
@techwritermat techwritermat deleted the CLEN-1290 branch June 14, 2023 10:30
@pubnub-release-bot
Copy link
Contributor

🛑 Build failed. Please check release build for details 🛑

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: done This issue is considered resolved.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.