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

Improve display of deployment parse errors #1426

Closed
nikku opened this issue Jul 8, 2019 · 1 comment · Fixed by #1515
Closed

Improve display of deployment parse errors #1426

nikku opened this issue Jul 8, 2019 · 1 comment · Fixed by #1515
Assignees
@barmac
Labels
deploy enhancement New feature or request ux
Milestone
M31

Comments

@nikku
Copy link
Member

nikku commented Jul 8, 2019 •
edited by philippfromme
Loading

Is your feature request related to a problem? Please describe.

We currently display deployment errors in the log. When dealing with validation errors (i.e. missing condition) the user should be able to get the relevant information from them in order to fix them.

screely-1568186722616

Possible solutions

  • Display human-readable error message as notification
  • For more details click notification to open log

Linked to Prodpad idea

Child of #1439
@nikku nikku added enhancement New feature or request ux backlog Queued in backlog deploy labels Jul 8, 2019
@nikku nikku added this to the M30 milestone Jul 8, 2019
@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Jul 8, 2019 — with bpmn-io-tasks
@barmac barmac mentioned this issue Jul 24, 2019
Closed
9 tasks
@barmac
Copy link
Collaborator

barmac commented Aug 8, 2019

For now, we can simply display the error message instead of stringified error.

@nikku nikku modified the milestones: M30, M31 Aug 12, 2019
@barmac barmac self-assigned this Sep 25, 2019
@barmac barmac added in progress Currently worked on and removed ready Ready to be worked on labels Sep 25, 2019 — with bpmn-io-tasks
barmac added a commit that referenced this issue Sep 26, 2019
@barmac
feat(deploy): display readable error message in log
1a4afff 
Closes #1426
@barmac barmac mentioned this issue Sep 26, 2019
Merged
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Sep 26, 2019
barmac added a commit that referenced this issue Sep 30, 2019
@barmac
feat(deploy): display readable error message in log
70e4899 
Closes #1426
barmac added a commit that referenced this issue Sep 30, 2019
@barmac
feat(deploy): display readable error message in log
b19d2fe 
Closes #1426
barmac added a commit that referenced this issue Oct 1, 2019
@barmac
feat(deploy): display readable error message in log
b0c7873 
Closes #1426
barmac added a commit that referenced this issue Oct 1, 2019
@barmac
feat(deploy): display readable error message in log
5e3b3c2 
Closes #1426
barmac added a commit that referenced this issue Oct 2, 2019
@barmac
feat(deploy): display readable error message in log
45fad0b 
Closes #1426
philippfromme pushed a commit that referenced this issue Oct 4, 2019
@barmac @philippfromme
feat(deploy): display readable error message in log
f21b8d0 
Closes #1426
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Oct 4, 2019
philippfromme pushed a commit that referenced this issue Oct 4, 2019
@barmac @philippfromme
feat(deploy): display readable error message in log
8340692 
Closes #1426
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@barmac barmac

Labels
deploy enhancement New feature or request ux
Projects
None yet
Milestone
M31
Development

Successfully merging a pull request may close this issue.

1426 improve error message formatting camunda/camunda-modeler
2 participants
@nikku @barmac