Skip to content

Add function to check connection state, handle goodbye and disconnect callback - #34

Merged
Mahad-10 merged 4 commits into
mainfrom
connected-function-disconnect-callback
Oct 27, 2025
Merged

Add function to check connection state, handle goodbye and disconnect callback#34
Mahad-10 merged 4 commits into
mainfrom
connected-function-disconnect-callback

Conversation

@Mahad-10

@Mahad-10 Mahad-10 commented Oct 9, 2025

Copy link
Copy Markdown
Member

No description provided.

@Mahad-10
Mahad-10 marked this pull request as draft October 9, 2025 15:03
@Mahad-10
Mahad-10 force-pushed the connected-function-disconnect-callback branch 3 times, most recently from a032dbd to 551bf42 Compare October 10, 2025 11:35
@Mahad-10
Mahad-10 marked this pull request as ready for review October 10, 2025 12:08
@Mahad-10
Mahad-10 requested a review from om26er October 10, 2025 12:08
@Mahad-10
Mahad-10 force-pushed the connected-function-disconnect-callback branch 4 times, most recently from 227d53b to 075e385 Compare October 13, 2025 14:16
Comment thread lib/session.ts Outdated
})();
}

setDisconnectCallback(callback: () => Promise<void>): void {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
setDisconnectCallback(callback: () => Promise<void>): void {
onDisconnect(callback: () => Promise<void>): void {

@om26er

om26er commented Oct 24, 2025

Copy link
Copy Markdown
Member

please rebase

@Mahad-10
Mahad-10 force-pushed the connected-function-disconnect-callback branch 2 times, most recently from 6323682 to 8ccda3d Compare October 27, 2025 07:40
@Mahad-10
Mahad-10 requested a review from om26er October 27, 2025 07:46
@Mahad-10
Mahad-10 force-pushed the connected-function-disconnect-callback branch from 8ccda3d to 41d64ea Compare October 27, 2025 10:57
@Mahad-10
Mahad-10 force-pushed the connected-function-disconnect-callback branch from 41d64ea to 47622c0 Compare October 27, 2025 10:59
@Mahad-10
Mahad-10 merged commit 109ad35 into main Oct 27, 2025
1 check passed
@Mahad-10
Mahad-10 deleted the connected-function-disconnect-callback branch October 27, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants