Feature #17653

Allow parameters and smart variable api to unhide values without changing the hidden_value flag

Added by Ori Rabin over 1 year ago. Updated 9 days ago.

Status:Closed
Priority:Normal
Assignee:Ori Rabin
Category:API
Target version:1.15.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4215, https://github.com/theforeman/foreman/pull/4105, https://github.com/theforeman/foreman/pull/4207

Description

In the UI it's possible to unhide a hidden value just to see it's value without editing the parameter.
The api currently doesn't have an option to show the value so hiding values in the api means they have to be edited to see them.
Adding an option to unhide the values just for display means they can be hidden unless some parameter is passed.
This will allow the api and hammer to act closer to the way the UI works.


Related issues

Related to Hammer CLI - Bug #17652: Hidden flag has no impact on smart variables when using H... Resolved 12/13/2016

Associated revisions

Revision 9643a2cb
Added by Ori Rabin over 1 year ago

Fixes #17653 - Allow all parameters api to unhide values

Revision 3b9d7933
Added by Ori Rabin over 1 year ago

Refs #17653 - add show_hidden to apidoc

Revision 38a9ed0c
Added by Ori Rabin over 1 year ago

Refs #17653 - fix typo in show_hidden_parameters and add tests

History

#1 Updated by Ori Rabin over 1 year ago

  • Related to Bug #17652: Hidden flag has no impact on smart variables when using Hammer CLI added

#2 Updated by The Foreman Bot over 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4105 added

#3 Updated by Ivan Necas over 1 year ago

  • Target version set to 1.10.3

#4 Updated by Ori Rabin over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal over 1 year ago

  • Legacy Backlogs Release (now unused) set to 209

#6 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/4207 added

#7 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/4215 added

#8 Updated by Ivan Necas over 1 year ago

  • Target version changed from 1.10.3 to 1.11.1

Also available in: Atom PDF