Page MenuHomePhabricator

Make various small quality-of-life improvements for Almanac properties
ClosedPublic

Authored by epriestley on Apr 11 2018, 3:41 PM.
Tags
None
Referenced Files
F13061745: D19342.id46253.diff
Fri, Apr 19, 7:55 PM
Unknown Object (File)
Fri, Apr 19, 5:39 AM
Unknown Object (File)
Fri, Apr 19, 2:53 AM
Unknown Object (File)
Fri, Apr 19, 12:53 AM
Unknown Object (File)
Sat, Mar 30, 4:45 PM
Unknown Object (File)
Sat, Mar 30, 4:45 PM
Unknown Object (File)
Sat, Mar 30, 4:23 PM
Unknown Object (File)
Mar 21 2024, 12:57 AM
Subscribers
None

Details

Summary

Depends on D19341. Ref T12414. Ref T13120.

  • Fix a bug where default-valued properties didn't get rendered in grey as they're supposed to (as a hint that the value isn't customized).
  • When resetting a builtin property won't do anything, visually disable the button as a hint.
  • Allow Services to specify properties on their Bindings.
  • Specify that repository bindings have a "protocol" property, so it becomes an explicit thing in the UI. Previously, you had to read the documentation to figure this out.
  • When editing bindings, use the EditField and its configuration if possible. This turns the "Protocol" property into a dropdown in the UI where you select between "http", "https" and "ssh".
  • Give the "protocol" binding a smart default based on the port number of the corresponding interface.
Test Plan
  • Viewed properties on Services, Devices and Bindings.
  • Saw them render sensibly, and grey out + grey button when a builtin value has a default setting.
  • Saw "Protocol" appear as a default property on repository cluster bindings and get a smart value.
  • Edited "protocol", got a nice dropdown.

Screen Shot 2018-04-11 at 8.26.16 AM.png (222×1 px, 23 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable