Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Feature/add basic hooks #38

Merged
merged 3 commits into from
Aug 25, 2023
Merged

Feature/add basic hooks #38

merged 3 commits into from
Aug 25, 2023

Conversation

kamil-stasiak
Copy link
Contributor

No description provided.

Copy link
Contributor

@Crackhoff Crackhoff left a comment

Choose a reason for hiding this comment

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

I'd like to see more of SDK usage here, even in minimal example

<div style={{ display: "flex", flexDirection: "column" }}>
<input value={token} onChange={(e) => setToken(() => e?.target?.value)} placeholder="token" />
<button
onClick={() => {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that an option to disconnect would be quite useful here. This is a matter of simple check on server status and swapping the function into useDisconnect() function. It's always nice to show it is viable in API

Copy link
Contributor

@Crackhoff Crackhoff left a comment

Choose a reason for hiding this comment

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

Everything working properly

@kamil-stasiak kamil-stasiak merged commit 9a69886 into main Aug 25, 2023
2 checks passed
@kamil-stasiak kamil-stasiak deleted the feature/add-basic-hooks branch August 25, 2023 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants