Add UI write support for CMP parameters #11754
Assignees
Labels
component:cmp
Config Management Plugin related issues
component:ui
User interfaces bugs and enhancements
enhancement
New feature or request
Milestone
Comments
crenshaw-dev
added
enhancement
|
Would you please assign this to me @crenshaw-dev ? |
|
@Marvin9 we typically don't assign issues. Just link back to this issue from a PR, and they'll be connected. :-) |
10 tasks
|
Hi @crenshaw-dev, I came here from #5734 since it’s said #11754 is its follow-up, but this issue doesn’t mention the CLI. Is there a dedicated issue for the CLI? Or is the modification of the CLI implied by this issue? |
crenshaw-dev
added a commit
that referenced
this issue
Mar 21, 2023
* #11602 fix : Object options menu truncated when selected in ApplicationListView. Signed-off-by: schakradari <[email protected]> * CMP parameter changes Signed-off-by: schakradari <[email protected]> * fix: pointers to param values Signed-off-by: Michael Crenshaw <[email protected]> better? Signed-off-by: Michael Crenshaw <[email protected]> fix silliness Signed-off-by: Michael Crenshaw <[email protected]> terrible hacks Signed-off-by: Michael Crenshaw <[email protected]> maybe better codegen Signed-off-by: Michael Crenshaw <[email protected]> fix tests Signed-off-by: Michael Crenshaw <[email protected]> * same prefix issue fixed Signed-off-by: schakradari <[email protected]> * fix for delete param name Signed-off-by: schakradari <[email protected]> * lint changes Signed-off-by: schakradari <[email protected]> * lint fix Signed-off-by: schakradari <[email protected]> * lint fix Signed-off-by: schakradari <[email protected]> * finalChanges Signed-off-by: schakradari <[email protected]> * Delete application-resource-list.tsx Not needed for this PR. Signed-off-by: schakrad <[email protected]> * added file which was deleted as it was not the change needed for this feature. Signed-off-by: schakradari <[email protected]> * refactored MapValuField and added fix for some edge cases Signed-off-by: schakradari <[email protected]> * Update application-resource-list.tsx Reverting the change as this is not related to this PR. Signed-off-by: schakrad <[email protected]> * Reverting the change in application-resource-list Signed-off-by: schakradari <[email protected]> * Showing application parameter values irrespective of parameter present or not in plugin.yaml Signed-off-by: schakradari <[email protected]> * fix for lint errors Signed-off-by: schakradari <[email protected]> * fix false source mismatch Signed-off-by: Michael Crenshaw <[email protected]> * fix equals Signed-off-by: Michael Crenshaw <[email protected]> * fix swagger doc Signed-off-by: Michael Crenshaw <[email protected]> * Tooltip description change. Signed-off-by: schakrad <[email protected]> * fixed lint Signed-off-by: schakrad <[email protected]> * CMP fix for empty array. Signed-off-by: schakrad <[email protected]> --------- Signed-off-by: schakradari <[email protected]> Signed-off-by: Michael Crenshaw <[email protected]> Signed-off-by: schakrad <[email protected]> Signed-off-by: schakrad <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
yyzxw
pushed a commit
to yyzxw/argo-cd
that referenced
this issue
Aug 9, 2023
* argoproj#11602 fix : Object options menu truncated when selected in ApplicationListView. Signed-off-by: schakradari <[email protected]> * CMP parameter changes Signed-off-by: schakradari <[email protected]> * fix: pointers to param values Signed-off-by: Michael Crenshaw <[email protected]> better? Signed-off-by: Michael Crenshaw <[email protected]> fix silliness Signed-off-by: Michael Crenshaw <[email protected]> terrible hacks Signed-off-by: Michael Crenshaw <[email protected]> maybe better codegen Signed-off-by: Michael Crenshaw <[email protected]> fix tests Signed-off-by: Michael Crenshaw <[email protected]> * same prefix issue fixed Signed-off-by: schakradari <[email protected]> * fix for delete param name Signed-off-by: schakradari <[email protected]> * lint changes Signed-off-by: schakradari <[email protected]> * lint fix Signed-off-by: schakradari <[email protected]> * lint fix Signed-off-by: schakradari <[email protected]> * finalChanges Signed-off-by: schakradari <[email protected]> * Delete application-resource-list.tsx Not needed for this PR. Signed-off-by: schakrad <[email protected]> * added file which was deleted as it was not the change needed for this feature. Signed-off-by: schakradari <[email protected]> * refactored MapValuField and added fix for some edge cases Signed-off-by: schakradari <[email protected]> * Update application-resource-list.tsx Reverting the change as this is not related to this PR. Signed-off-by: schakrad <[email protected]> * Reverting the change in application-resource-list Signed-off-by: schakradari <[email protected]> * Showing application parameter values irrespective of parameter present or not in plugin.yaml Signed-off-by: schakradari <[email protected]> * fix for lint errors Signed-off-by: schakradari <[email protected]> * fix false source mismatch Signed-off-by: Michael Crenshaw <[email protected]> * fix equals Signed-off-by: Michael Crenshaw <[email protected]> * fix swagger doc Signed-off-by: Michael Crenshaw <[email protected]> * Tooltip description change. Signed-off-by: schakrad <[email protected]> * fixed lint Signed-off-by: schakrad <[email protected]> * CMP fix for empty array. Signed-off-by: schakrad <[email protected]> --------- Signed-off-by: schakradari <[email protected]> Signed-off-by: Michael Crenshaw <[email protected]> Signed-off-by: schakrad <[email protected]> Signed-off-by: schakrad <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
v2.6 includes a new way to parameterize Applications that use config management plugins.
When a plugin "advertises" a certain parameter, that parameter appears in the UI under the Application's "Parameters" tab.
That view is currently read-only for those parameters. The UI should allow a user to edit the values. The UI should transform the value as presented in the UI into the
spec.source.plugin.parametersformat.What it looks like as a read-only interface:
What it should look like in the manifest after editing:
(Look under
source.plugin.parameters.)Motivation
This will significantly ease the user experience of setting plugin parameters.
Proposal
Mimic the current Helm and Kustomize parameters edit experience as much as possible.
The text was updated successfully, but these errors were encountered: