Page MenuHomeDevCentral
Differential D175

ActionsReport::addAction
ClosedPublic
Actions

Authored by dereckson on Jan 8 2016, 13:36.
Tags
None
Referenced Files
F3912460: D175.diff
Thu, Dec 19, 05:56
Unknown Object (File)
Sat, Dec 14, 18:53
Unknown Object (File)
Fri, Dec 13, 04:19
Unknown Object (File)
Thu, Dec 5, 00:56
Unknown Object (File)
Wed, Dec 4, 18:35
Unknown Object (File)
Tue, Dec 3, 01:15
Unknown Object (File)
Mon, Dec 2, 09:00
Unknown Object (File)
Mon, Dec 2, 02:40
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
Unknown Object (Diffusion Commit)
rNOTIF848b401fb498: ActionsReport::addAction
Summary

We want to allow to decouple the report class, a generic object, and
the events listener, specific to the report the application emits.

For this purpose, we add a new public method, ActionsReport::addAction.

Test Plan

Send a payload and ensure the report contains actions.

Unit tests to come in D176.

Diff Detail

Repository
rNOTIF Notifications center
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
ActionsReport
Build Status
Buildable 218
Build 218: arc lint + arc unit

Event Timeline

dereckson updated this revision to Diff 415.Jan 8 2016, 13:36
dereckson retitled this revision from to ActionsReport::addAction.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson accepted this revision.Jan 8 2016, 13:37
dereckson edited edge metadata.
This revision is now accepted and ready to land.Jan 8 2016, 13:37
Closed by commit rNOTIF848b401fb498: ActionsReport::addAction (authored by dereckson, committed by dereckson). · Explain WhyJan 8 2016, 13:37
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
app/
Actions/
11 lines

Diff 415

View Options

app/Actions/ActionsReport.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator