In T13269#245817, @pasik wrote:well, I'd say for example the "Close as <status> (only acts if task is not already closed)" is useful, and definitely needed.
Move the task to "done" column on the workboard, and the task automatically gets closed. That's handy.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Apr 10 2019
Apr 10 2019
amckinley added a comment to T13269: Improve initial implementations of Workboard triggers and groups.
amckinley committed rP55d64d0fabd3: Add trigger rule to remove projects from tasks (authored by amckinley).
Add trigger rule to remove projects from tasks
amckinley committed rP8b475898ee92: Add trigger rule for adding projects to a task (authored by amckinley).
Add trigger rule for adding projects to a task
ok this time i got it
Oops.
Change rule to unconditionally add projects in transaction and let the editor figure it out.
amckinley added a comment to T13269: Improve initial implementations of Workboard triggers and groups.
Broadly, it's not clear to me that any of these rules are actually good/useful/worth building.
amckinley added a comment to T13269: Improve initial implementations of Workboard triggers and groups.
- Close as <status> (only acts if task is not already closed).
- Reopen as <status>.
amckinley updated the task description for T13269: Improve initial implementations of Workboard triggers and groups.
amckinley accepted D20387: Surface custom form instructions as a "Remarkup" field for the transaction layer.
BUT WHY DID YOU CROP THE PICTURE
Apr 9 2019
Apr 9 2019
We probably can't really run this "alongside" the current production git endpoint without doing a ton of work, can we? It would be nice to watch this thing have byte-identical output to prod for a while before cutting over.
Apr 8 2019
Apr 8 2019
Apr 5 2019
Apr 5 2019
Add a trigger rule to reassign a task
Change method name, remove extraneous argument.
Apr 4 2019
Apr 4 2019
I found two existing bugs while writing this:
This was about 100x more complicated than I expected because I went around and around about how to handle getting the viewer context down to the level of the rule objects. It turns out there are quite a few ways to generate trigger and rule objects, and some of them make more sense than others for explicitly passing $viewer around. Settled on manually setting the viewer inside the controllers that need to work with triggers.
Pretty much everything works now, including drop action previews. Only breakage is UI stuff when changing the type of an existing trigger rule.
amckinley added a comment to D20377: When a dashboard has inconsistent panel policies: keep doing nothing.
It's possible that we don't really need this anymore -- at the time, I think one panel immediately banished the whole board to the shadow realm, while everything degrades pretty cleanly now.
amckinley added a comment to D20377: When a dashboard has inconsistent panel policies: keep doing nothing.
We'd have to make an assertion like "panels must have the same policy as the dashboard, or a strictly weaker policy".
Apr 3 2019
Apr 3 2019
Ok this makes more sense now that I've looked at the follow up revisions.
Wait, we have both of PhabricatorDashboardPanelEditproController and PhabricatorDashboardPanelEditController, and they don't have the same code, and we're keeping both of them? Did you write one, forget about it, couldn't find it because of the incorrect name, and write it again?
Apr 2 2019
Apr 2 2019
amckinley committed rP3e05ff2e9983: Improve Conpherence behavior for logged out users. (authored by amckinley).
Improve Conpherence behavior for logged out users.
Apr 1 2019
Apr 1 2019
amckinley added a comment to D20350: Retry connections on timeouts, and raise more readable connection failure messages.
I skimmed the other client-side error codes, and the only other condition that stood out as being another retry candidate was 2013, "lost connection to server". I'm happy to say "don't add it until someone asks for it", though.
amckinley accepted D20350: Retry connections on timeouts, and raise more readable connection failure messages.
The code looks pretty obviously correct to me, but maybe test the other case to make sure we aren't suddenly retrying for all error codes? Try, uh, error 2005, "Unknown MySQL server host"?
Mar 29 2019
Mar 29 2019
Mar 28 2019
Mar 28 2019
While you're here, have you taken a look at T13273?
Mar 27 2019
Mar 27 2019
In D20335#255596, @epriestley wrote:I think knowing how to refine the query might be helpful, especially for more technical users.
amckinley accepted D20333: Use real Dashboard Panels to render the default hard-coded homepage, not hacky fake panels.
Mar 26 2019
Mar 26 2019
In D20329#255482, @amckinley wrote:Makes preview behavior work correctly. Still doesn't quite do the right thing: the value is getting mangled when the rules come out of the DB, but not on the way in.
Makes preview behavior work correctly. Still doesn't quite do the right thing: the value is getting mangled when the rules come out of the DB, but not on the way in.