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
[Enhancement] Show an error notification for errors in deck #1373
Conversation
igorDykhta
commented
Dec 10, 2020
•
edited
edited
- custom _onDeckError handler in map-container with error throttling to prevent multiple similar errors from appearing.
- additional logic for addNotification to update existing notifications.
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.
LGTM though I am not kepler approver.
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
a233c0a
to
28ddc0f
Compare
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
5b7bfa3
to
a2fee8b
Compare
@@ -251,6 +256,28 @@ export default function MapContainerFactory(MapPopover, MapControl, Editor) { | |||
setLayerBlending(gl, this.props.layerBlending); | |||
}; | |||
|
|||
_onError = (error, layer) => { | |||
const errorMessage = `An error in deck.gl: ${error.message} in ${layer.id}`; |
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.
Will error
from deck.gl always have a message?
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.
Good question.
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
3e4019f
to
412c4c7
Compare
Signed-off-by: Igor Dykhta <dikhta.igor@gmail.com>
ebee7bc
to
38c5a8f
Compare