Skip to content
This repository has been archived by the owner on Nov 25, 2022. It is now read-only.

Error handlers #8

Closed
wants to merge 1 commit into from
Closed

Error handlers #8

wants to merge 1 commit into from

Conversation

Fofir
Copy link

@Fofir Fofir commented Mar 16, 2017

  • Added the onPublisherInitError and onPublishError as props
    for . These wil be triggered when the publisher
    fails to initiate and when the publisher fails to publish
    repectively

  • Added the onSessionConnectError property for the createSession
    helper. Will be triggered is session.connect fails

@aiham - as we discussed in #7, this is just from the top of my mind. Let me know what you think and if that is the direction in which you were thinking!

+ Added the `onPublisherInitError` and `onPublishError` as props
for <OTPublisher/>. These wil be triggered when the publisher
fails to initiate and when the publisher fails to publish
repectively

+ Added the `onSessionConnectError` property for the `createSession`
helper. Will be triggered is `session.connect` fails
@Fofir Fofir changed the title Initial commit of error handlers Error handlers Mar 16, 2017
@aiham
Copy link
Contributor

aiham commented Mar 16, 2017

Awesome thanks. Let me think about this a bit.

@aiham
Copy link
Contributor

aiham commented Mar 25, 2017

#10 preferred

@aiham aiham closed this Mar 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants