This repository has been archived by the owner. It is now read-only.

don’t call doAction inside doAction because it has the potential to l… #6885

Merged

bbondy merged 1 commit into master from do-action

Jan 27, 2017

+2 −2

Collaborator

bridiver commented Jan 27, 2017

…ose updates

Submitted a ticket for my issue if one did not already exist.
Used Github auto-closing keywords in the commit message.
Added/updated tests for this change (for new code or code which already has tests).
Ran git rebase -i to squash commits (if needed).

Test Plan:


        ensure that the menu close action happens after the click action

5f7891a

fix #6791

bridiver force-pushed the do-action branch from 7968884 to 5f7891a

bbondy merged commit 0fe40d8 into master

0 of 3 checks passed

0 of 3 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete

Details

continuous-integration/travis-ci/pr The Travis CI build is in progress

Details

continuous-integration/travis-ci/push The Travis CI build is in progress

Details

luixxiul added this to the 0.13.0 milestone

bsclifton deleted the do-action branch

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

You can’t perform that action at this time.