-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FCE-890] Update documentation to use new useConnection
hook
#54
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I think we should also update |
@mironiasty @MiloszFilimowski I have updated the docs for |
@@ -46,7 +46,8 @@ follow our [server setup instructions](/production/server). | |||
In order to connect, you just need to call `joinRoom` method with data from previous step: | |||
|
|||
```ts | |||
import { joinRoom } from "@fishjam-cloud/react-native-client"; | |||
import { useConnection } from "@fishjam-cloud/react-native-client"; | |||
const { joinRoom } = useConnection(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: Can you add a new line after the import? 🙏
@@ -56,7 +57,8 @@ await joinRoom(fishjamUrl, peerToken); | |||
In order to close connection, you have to call `leaveRoom` method. | |||
|
|||
```ts | |||
import { leaveRoom } from "@fishjam-cloud/react-native-client"; | |||
import { useConnection } from "@fishjam-cloud/react-native-client"; | |||
const { leaveRoom } = useConnection(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: Same request 🙏
@MiloszFilimowski Done! |
Description
Changed
usePeerStatus
anduseReconnection
touseConnection
.