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
F19090353: D19342.id.diff
Dec 3 2025, 11:17 AM
F19086821: D19342.diff
Dec 2 2025, 10:45 PM
F19066369: D19342.id46253.diff
Nov 30 2025, 5:11 AM
F18829859: D19342.diff
Oct 25 2025, 1:40 AM
F18766578: D19342.diff
Oct 7 2025, 5:25 PM
F18746762: D19342.id.diff
Oct 3 2025, 1:23 PM
F18746360: D19342.id46280.diff
Oct 3 2025, 12:14 PM
F18702204: D19342.id46253.diff
Sep 27 2025, 11:10 PM
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