ActionsReport::addAction
ClosedPublic
Actions

Authored by dereckson on Jan 8 2016, 13:36.

Tags

None

Referenced Files

	F25050521: D175.diff
	Fri, Mar 27, 10:25

	Unknown Object (File)
	Thu, Mar 26, 22:30

	Unknown Object (File)
	Thu, Mar 26, 12:23

	Unknown Object (File)
	Thu, Mar 26, 04:09

	Unknown Object (File)
	Wed, Mar 25, 09:18

	Unknown Object (File)
	Tue, Mar 17, 05:59

	Unknown Object (File)
	Sun, Mar 15, 23:54

	Unknown Object (File)
	Sun, Mar 15, 22:00

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

Lint Not Applicable

Unit

Tests Not Applicable

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

Path

Size

app/

Actions/

ActionsReport.php

11 lines

Diff 416

View Options

app/Actions/ActionsReport.php

ActionsReport::addActionClosedPublicActions