Skip to content

Better Geo Node Error Messages MVC

Zack Cuddy requested to merge 213363-better-node-errors into master

What does this MR do?

Closes #213363 (closed)

Currently in the UI we have a generic error message when the Geo Node Form errors. However, the API returns string errors that provide context to why it failed.

This MR simply exposes the error messages to the flash message so the user has some idea of what went wrong.

Important Notes

This is an MVC to begin bringing valuable errors to the end user.

I am creating a follow up issue to talk about making the errors more user friendly and to gather ideas on how to do that. This MR was simply to expose the information.

Screenshots

error
before Error
after Screen_Shot_2020-04-16_at_9.46.25_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Closes #213363 (closed)

Edited by Zack Cuddy

Merge request reports