-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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: add ability to edit pki key #18490
Conversation
hellobontempo
commented
Dec 20, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
@@ -1,5 +1,5 @@ | |||
<label for={{@attr.name}} class="is-label" data-test-readonly-label> | |||
{{this.labelString}} | |||
{{or @label this.labelString}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: debating whether this should be in the getter for this.labelString
. On one hand, this makes it very clear that passing @label
in explicitly will override any other calculations. But, we already have a method here for calculating the label and it could easily live there 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had it that way initially and and thought the getter had a lot going on as well - but I'm happy to move it back there!
Another thought is I could do something similar to the above, and create a let
for keyType
and leave this component alone. This also gives us the benefit of including the option's subtext... 🤔 Any opinion?
{{/each}} | ||
{{else}} | ||
{{! only keyName is edit-able }} | ||
{{#let (find-by "name" "keyName" @model.formFields) as |attr|}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤩
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy
* add edit form * refactor readonly field * refactor conditional in key form * remove model validations from edit form * update namespace reminder opy