search

boomerang-io / community

The Boomerang community, roadmap, planning, and architecture repository. The central place for information on joining, contributing, and governance.
https://useboomerang.io
Apache License 2.0
11 stars 0 forks source link

UI + Workflow - Validate Secure Parameters #413

Closed tlawrie closed 2 weeks ago

tlawrie commented 1 year ago

Secure parameters correctly have the value and defaultValue fields wiped out.

However when an update comes through, they aren't merged, and hence the blank value / defaultValue gets stored and the value wiped out.

Need to update all the places the merge happens with a common method.

Relates to https://github.com/boomerang-io/community/issues/437 - if we combine how config and params are stored, it will make this a lot easier.

tlawrie commented 2 weeks ago

Fixed.