Feature #17653

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

Added by Ori Rabin 6 months ago. Updated 5 months ago.

Status:Closed
Priority:Normal
Assigned To:Ori Rabin
Category:API
Target version:Team Ivan Iteration 9
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4215, https://github.com/theforeman/foreman/pull/4207, https://github.com/theforeman/foreman/pull/4105
Story points-
Velocity based estimate-
Release1.15.0Release relationshipAuto

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 5 months ago

Fixes #17653 - Allow all parameters api to unhide values

Revision 3b9d7933
Added by Ori Rabin 5 months ago

Refs #17653 - add show_hidden to apidoc

Revision 38a9ed0c
Added by Ori Rabin 5 months ago

Refs #17653 - fix typo in show_hidden_parameters and add tests

History

#1 Updated by Ori Rabin 6 months ago

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

#2 Updated by The Foreman Bot 6 months ago

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

#3 Updated by Ivan Necas 6 months ago

  • Target version set to Team Ivan Iteration 8

#4 Updated by Ori Rabin 5 months ago

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

#5 Updated by Dominic Cleal 5 months ago

  • Release set to 1.15.0

#6 Updated by The Foreman Bot 5 months ago

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

#7 Updated by The Foreman Bot 5 months ago

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

#8 Updated by Ivan Necas 5 months ago

  • Target version changed from Team Ivan Iteration 8 to Team Ivan Iteration 9

Also available in: Atom PDF