Skip to content

Geo Node Form in Vue - Core Fields

Zack Cuddy requested to merge 118841_02-node-form-in-vue into master

Important Notes

This is the an MR broken off of !22719 (closed) This will allow for the MR to be easier to read and review without the giant MR all in one.

Work is towards the goal of #118841 (closed)

What does this MR do?

This MR creates the Geo Node Form in Vue with the Core fields.

Significant Note: The selective sync fields will be done in the next MR. Note the differences in the Secondary screenshot

This MR works behind the feature flag: :enable_geo_node_form_js

This feature flag will be removed as part of closing #118841 (closed)

This MR itself is an attempt at a MVC as well as to keep the MRs small around this overall change.

The end goal is to transition the Geo Node form into Vue along with additional form validations and updated styles.

Screenshots

Node Form Primary Node Form Secondary
before Geo_Node_Form_-_Primary__Old Geo_Node_Form_-_Secondary__Old
after Primary_Form_New Secondary_Form_New

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
Edited by Zack Cuddy

Merge request reports