Page MenuHomePhabricator

Modularize Dashboard Panel transactionns
ClosedPublic

Authored by epriestley on Apr 2 2019, 5:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 31 2024, 6:24 PM
Unknown Object (File)
Jan 6 2024, 2:11 PM
Unknown Object (File)
Dec 26 2023, 11:44 PM
Unknown Object (File)
Dec 20 2023, 11:33 AM
Unknown Object (File)
Dec 17 2023, 10:18 AM
Unknown Object (File)
Dec 7 2023, 7:03 PM
Unknown Object (File)
Nov 30 2023, 3:23 AM
Unknown Object (File)
Nov 16 2023, 8:59 AM
Subscribers
None

Details

Summary

Depends on D20369. Ref T13272. Move toward a world where we can edit panels with just one controller, instead of separate "Edit" and "Editpro" controllers.

Test Plan

Created and edited panels. This will get vetted more thoroughly after additional changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/dashboard/xaction/panel/PhabricatorDashboardPanelNameTransaction.php
64–69

Isn't this the default implementation for ModularTransactions?

This revision is now accepted and ready to land.Apr 3 2019, 6:53 PM
src/applications/dashboard/xaction/panel/PhabricatorDashboardPanelNameTransaction.php
64–69

By default, we don't expose anything to Conduit.

Some transactions have complex/weird/sensitive/not-good-for-API-users details, and if we exposed them by default we might end up in trouble (maybe users trying to use them and complaining that they're impossible to work with; maybe users complaining that we broke compatibility if we then fix it; maybe actual security problems), so we err on the side of caution and require each transaction have an explicit implementation.

In this case (and other simple cases) there's an obviously reasonable way to represent the transaction, but I just delete / don't implement these methods a lot of the time if there's any ambiguity.

Usually, this only impacts webhooks trying to do something with the event.

This revision was automatically updated to reflect the committed changes.