Page MenuHomePhabricator

Cleanup some clustering rough edges
ClosedPublic

Authored by amckinley on Dec 20 2018, 12:26 AM.
Tags
None
Referenced Files
F13078806: D19917.diff
Tue, Apr 23, 6:47 PM
Unknown Object (File)
Wed, Apr 17, 3:04 PM
Unknown Object (File)
Thu, Apr 11, 7:14 AM
Unknown Object (File)
Mar 10 2024, 11:55 AM
Unknown Object (File)
Jan 9 2024, 9:20 AM
Unknown Object (File)
Dec 24 2023, 3:40 PM
Unknown Object (File)
Dec 19 2023, 8:45 AM
Unknown Object (File)
Dec 13 2023, 2:33 AM
Subscribers

Details

Summary

Suppress an unhelpful Almanac transaction and document the location of the secret clustering management capability. I thought maybe implementing shouldHide and checking for isCreate would work, but the binding apparently gets created before an interface is bound to it.

Test Plan

Looked at a fresh binding and didn't see "Unknown Object(??)", ran bin/diviner and saw expected output.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/almanac/xaction/AlmanacBindingInterfaceTransaction.php
59–62

Some day, this transaction will be marked as TYPE_CREATE and hidden, but adding a better rendering to it is quite reasonable for now (and maybe we'll go back and add more of these if we let you render hidden transactions with transaction.search at some point).

63–66

I think removing an interface is impossible so this code is unreachable.

This revision is now accepted and ready to land.Dec 20 2018, 2:53 PM
This revision was automatically updated to reflect the committed changes.