Skip to content

Migrate Edit Environments Form to Vue

Andrew Fontaine requested to merge afontaine/vueify-edit-environment into master

What does this MR do?

Re-uses the new form component made as part of the migration of new environments.

There was some question about whether or not we need Vuex or something, but honestly because the form manages such a low number of properties and the validation requirements are so small, that I opted to not use it here instead.

Also the environment model is small, and so can be inlined in a data attribute. As the validation is stricter on the backend, this should be clear of HTML injection.


Once GraphQL becomes an option here, then it would make a lot of sense to just use Apollo to manage most everything I think.

For #29006 (closed)

Screenshots or Screencasts (strongly suggested)

image

image

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Andrew Fontaine

Merge request reports

Loading