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.

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

Ui/transform sidebranch #9665

Merged
merged 22 commits into from
Aug 26, 2020
Merged

Ui/transform sidebranch #9665

merged 22 commits into from
Aug 26, 2020

Conversation

chelshaw
Copy link
Contributor

@chelshaw chelshaw commented Aug 5, 2020

Initial work for the Transform Secret Engine, which includes:

  • Enable Transform Secrets Engine
  • List Transformations
  • Show Transformation detail
  • Edit Transformation
  • Create New Transformation

Enable (with ADP module)
secret-mounts-with-adp

Enable (without ADP module
secret-mounts-without-adp

Empty view of transformations
transformations-empty

Create transformation
create-transformation

List transformations
list-transformations

Edit transformation
edit-transformation

chelshaw and others added 2 commits July 31, 2020 14:09
* Show Transform on engines list if enterprise

* Add box-radio component

* Add is-disabled styling for box-radio and fix tooltip styling when position: above

* Add KMIP and Transform to possible features on has feature helper
* WIP // list transforms, console.logs and all

* setup LIST transformations ajax request and draft out options-for-backend options

* change from plural to singluar and add transform to secret-edit

* create two transform edit components

* modify transform model with new attrs

* add adapterFor to connect transform adapter to transform-edit-form component

* setup Allowed roles searchSelect component to search over new transform/role adapter and model.

* clean up for PR

* clean up linting errors

* restructure adapter call, now it works.

* remove console

* setup template model for SearchSelect component

* add props to form field and search select for styling

Co-authored-by: Chelsea Shaw <[email protected]>
@chelshaw chelshaw added this to the 1.6 milestone Aug 5, 2020
chelshaw and others added 5 commits August 10, 2020 15:07
* Update casing and wording on Transform list route. Use generic list item for transformations

* Add back js file for transformation-edit

* Set up transform for tabs
* add conditional for masking vs tweak source based on type, and update text for create transformation

* change order

* fix error with stringArray

* setup the edit/delete transformation view

* clean up toolbar links

* setup serializer to change response of mask character from keycode to character

* change styling of label and sub-text size, confirmed with design

* temp fix on templates vs template

* add clickable list item

* add space between template list

* setup styling and structure for the rest of the show transformation.  TODO: turn into components.

* create transform-show-transformation component

* add attachCapabilities to transform model and update transform-transformation-itme list accordingly

* clean up liniting errors

* address pr comments

* remove leftover

* clean up
* clean up some of the TODOs

* setup edit view with read only attributes for name and template

* setup initial selected for search select component

* fixes

* hide templates form field for now

* set selectLimit for search select component

* hide power select if the select limit is greater than or equal to the selectedOptions length

* clean up failing linting

* address pr comments
@chelshaw chelshaw marked this pull request as ready for review August 19, 2020 18:26
chelshaw and others added 4 commits August 20, 2020 11:43
* Update search-select to pass backend to query if exists

* Update role and template adapters

* cleanup

* Fix replace with static string
* amend encode/decode commands for now until design gets back with more details

* restrict character count on masking input field

* clean up selectLimit

* show backend instead of transform in cli copy command
@Monkeychip Monkeychip requested a review from a team August 20, 2020 19:24
@hashicorp hashicorp deleted a comment Aug 20, 2020
@hashicorp hashicorp deleted a comment Aug 20, 2020
Copy link
Contributor

@andaley andaley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is looking great! all of the comments/questions i had are non-blocking. one suggestion to make this PR more robust would be to add an acceptance test to ensure that all of the critical workflows (i.e. creating / editing a transform) work end to end.

nice work! 🤠

ui/app/components/transform-edit-form.js Outdated Show resolved Hide resolved
ui/app/components/transform-show-transformation.js Outdated Show resolved Hide resolved
ui/app/helpers/options-for-backend.js Show resolved Hide resolved
ui/app/models/transform.js Show resolved Hide resolved
ui/app/models/transform.js Show resolved Hide resolved
<form onsubmit={{action "createOrUpdate" "create"}}>
<div class="box is-sideless is-fullwidth is-marginless">
{{message-error model=model}}
{{!-- TODO: figure out what this ?? --}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, this might be a reminder that editing something here only applies to your namespace? that's my wild guess, at least

>
{{#if (eq mode 'create')}}
Create transformation
{{else if (eq mode 'edit')}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if this is the edit form, why/when would the mode be create? and similarly, why would the TransformCreateForm's mode ever be edit?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a case of unfortunate naming in the secrets engine space. The component name for show/create/edit is saved as editComponent in options-for-backend, and we're following the rest of the naming conventions here. I can see the value of keeping it consistent with the others, as well as the value of naming it something more specific like transform-show-create-edit.hbs (although that's quite verbose) 🤔

ui/lib/core/addon/components/search-select.js Outdated Show resolved Hide resolved
ui/lib/core/addon/templates/components/form-field.hbs Outdated Show resolved Hide resolved
@chelshaw chelshaw merged commit 6fb9768 into master Aug 26, 2020
@chelshaw chelshaw deleted the ui/transform-sidebranch branch August 26, 2020 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants