Geo Node Form in Vue - Core Fields
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 | ||
after |
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation (if required) -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers
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