Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

Send pull request notifications asynchronously when possible

Download Browse

Pull requests notifications are sent asynchronously as soon as it is possible to get a Redis persistent queue, otherwise it fallbacks to a synchronous mode. Part of story #14190: get email notifications on pull requests - first stage Change-Id: I67790d6f8abd0615c10d22cd0f362cce10925d87

+585 −67
Thomas Gerbet (tgerbet) 2019-12-05 18:10
Thomas Gerbet (tgerbet) 2019-12-06 14:15
9a65ce7e0f844c35ec5b78419f8dd2c8e45ce027
73d8d896725fb6df546a0f2b86859302ed837c5f
git #tuleap/stable/9a65ce7e0f844c35ec5b78419f8dd2c8e45ce027

Modified Files

Side by side diff
Inline diff
Name
M plugins/pullrequest/include/PullRequest/Authorization/PullRequestPermissionChecker.php +6 −7 Go to diff View file
A plugins/pullrequest/include/PullRequest/Notification/EventDispatcherWithFallback.php +69 −0 Go to diff View file
M plugins/pullrequest/include/PullRequest/Notification/EventSubjectToNotification.php +4 −0 Go to diff View file
A plugins/pullrequest/include/PullRequest/Notification/EventSubjectToNotificationAsynchronousRedisDispatcher.php +64 −0 Go to diff View file
A plugins/pullrequest/include/PullRequest/Notification/InvalidWorkerEventPayloadException.php +33 −0 Go to diff View file
A plugins/pullrequest/include/PullRequest/Notification/PullRequestNotificationSupport.php +129 −0 Go to diff View file
M plugins/pullrequest/include/PullRequest/REST/v1/PullRequestsResource.php +15 −52 Go to diff View file
M plugins/pullrequest/include/PullRequest/Reviewer/Change/ReviewerChangeEvent.php +18 −1 Go to diff View file
M plugins/pullrequest/include/pullrequestPlugin.php +14 −1 Go to diff View file
M plugins/pullrequest/phpunit/Authorization/PullRequestPermissionCheckerTest.php +7 −6 Go to diff View file
A plugins/pullrequest/phpunit/Notification/EventDispatcherWithFallbackTest.php +98 −0 Go to diff View file
A plugins/pullrequest/phpunit/Notification/EventSubjectToNotificationAsynchronousRedisDispatcherTest.php +78 −0 Go to diff View file
M plugins/pullrequest/phpunit/Notification/EventSubjectToNotificationListenerProviderTest.php +4 −0 Go to diff View file
A plugins/pullrequest/phpunit/Reviewer/Change/ReviewerChangeEventTest.php +46 −0 Go to diff View file