Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Better RPD Model Error system #13

Closed
shamansir opened this issue Jan 8, 2015 · 5 comments · Fixed by #350
Closed

Better RPD Model Error system #13

shamansir opened this issue Jan 8, 2015 · 5 comments · Fixed by #350
Labels
could-change-api
Milestone
v2.0.x

Comments

@shamansir
Copy link
Owner

Show them clearly in UI, and skip them if possible.
@shamansir
Copy link
Owner Author

Probably the same as #12, since Kefir now supports errors in streams.

@shamansir
Copy link
Owner Author

All of "system" errors (like unregistered inlet type) should be thrown as JS errors and then shown in UI somewhere at the top... may be

@shamansir shamansir assigned shamansir and unassigned shamansir Jan 30, 2015
@shamansir shamansir modified the milestone: 0.2.0 Feb 1, 2015
@shamansir shamansir modified the milestones: v0.2.0, v0.2.5 Jul 20, 2015
@shamansir
Copy link
Owner Author

Showing errors: #164

@shamansir
Copy link
Owner Author

All of "system" errors (like unregistered inlet type) should be thrown as JS errors and then shown in UI somewhere at the top... may be

Wrong, errors should be fired as Kefir errors in streams and passed to parent collecting pool.

@shamansir shamansir mentioned this issue Aug 26, 2015
Open
@shamansir shamansir modified the milestones: v0.2.0-β, v0.2.0-α Dec 29, 2015
@shamansir shamansir mentioned this issue Feb 10, 2016
Open
@shamansir shamansir mentioned this issue Mar 31, 2016
Closed
@shamansir
Copy link
Owner Author

errors should be fired as Kefir errors in streams and passed to parent collecting pool.

...and then, show them in UI

@shamansir shamansir modified the milestones: v0.2.0-α, v0.2.0-β Mar 31, 2016
@shamansir shamansir mentioned this issue Mar 31, 2016
Merged
5 tasks
@shamansir shamansir mentioned this issue Apr 1, 2016
Open
shamansir added a commit that referenced this issue Apr 1, 2016
@shamansir
pass subject to the fired error (relates to #13, #331)
7bdd992
shamansir added a commit that referenced this issue May 6, 2016
@shamansir
log system errors as errors and usual errors as normal log (#13, #331, …
1b6df19 
…#164)
shamansir added a commit that referenced this issue May 6, 2016
@shamansir
fix error reporting for channel.accept (#315, #164, #12, #13)
0bd7ae6
shamansir added a commit that referenced this issue May 8, 2016
@shamansir
remove error logging from rpd.js, move it to renderer and allow to co…
0f7d954 
…nfigure it (#164, #13, #315)
@shamansir shamansir mentioned this issue May 20, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
could-change-api
Projects
None yet
Milestone
v2.0.x
Development

Successfully merging a pull request may close this issue.

Docs, Node Type List as a Node, Better Error Handling and Node Type Icons shamansir/rpd
1 participant
@shamansir