Page MenuHomePhabricator

Allow Nuance commands to try to apply immediately
ClosedPublic

Authored by epriestley on May 24 2017, 8:27 PM.
Tags
None
Referenced Files
F13091744: D18015.diff
Thu, Apr 25, 3:11 AM
Unknown Object (File)
Mon, Apr 22, 9:18 AM
Unknown Object (File)
Mon, Apr 22, 9:17 AM
Unknown Object (File)
Mon, Apr 22, 9:14 AM
Unknown Object (File)
Mon, Apr 22, 9:00 AM
Unknown Object (File)
Fri, Apr 19, 6:32 PM
Unknown Object (File)
Wed, Apr 10, 10:41 PM
Unknown Object (File)
Feb 18 2024, 10:42 PM
Subscribers

Details

Summary

Ref T12738. By default, we process Nuance commands in the background. The intent is to let the user continue working at full speed if Twitter or GitHub (or whatever) is being a little slow.

Some commands don't do anything heavy and can be processed in the foreground. Let commands choose to try foreground execution.

Test Plan

Threw complaints in the trash, saw them immediately go into the trash.

Diff Detail

Repository
rP Phabricator
Branch
nuance9
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/nuance/controller/NuanceItemActionController.php:99XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 17241
Build 23085: Run Core Tests
Build 23084: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.May 24 2017, 8:38 PM

mmm

Anecdotally, In T9530 we had a concept of "Actions" that users can take on Releases - e.g. "Move to the next phase in the cycle". We broadly wanted to use Harbormaster to allow defining these actions, and one of the issues I had there was that for some actions, the users expected them to happen inline, pretty much like here.

This revision was automatically updated to reflect the committed changes.