Page MenuHomePhabricator

Cleanup some clustering rough edges
ClosedPublic

Authored by amckinley on Dec 20 2018, 12:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 10:27 PM
Unknown Object (File)
Sep 9 2024, 11:33 PM
Unknown Object (File)
Sep 2 2024, 9:32 AM
Unknown Object (File)
Aug 29 2024, 2:16 PM
Unknown Object (File)
Aug 27 2024, 11:10 PM
Unknown Object (File)
Aug 24 2024, 8:35 PM
Unknown Object (File)
Aug 21 2024, 6:59 PM
Unknown Object (File)
Aug 17 2024, 12:54 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
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21393
Build 29131: Run Core Tests
Build 29130: arc lint + arc unit

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.