Set all room settings at once
ClosedPublic
Actions

Authored by chad on Apr 19 2017, 9:21 PM.

Tags

None

Referenced Files

	F19025640: D17735.id.diff
	Mon, Nov 24, 8:19 AM

	F19020841: D17735.diff
	Sun, Nov 23, 6:22 PM

	F18873861: D17735.id.diff
	Wed, Nov 5, 11:48 AM

	F18816086: D17735.id.diff
	Oct 21 2025, 4:52 AM

	F18810002: D17735.diff
	Oct 19 2025, 6:43 PM

	F18796463: D17735.diff
	Oct 17 2025, 3:33 AM

	F18679160: D17735.id.diff
	Sep 26 2025, 12:33 AM

	F18627235: D17735.id.diff
	Sep 16 2025, 3:32 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rPdb60af7ea57c: Set all room settings at once

Summary

Sets notification and sound preferences in a single array()

Test Plan

Change email preference, save, set sound preference, save. Email preference still OK.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Apr 19 2017, 9:21 PM

Herald added a subscriber: Korvin. · View Herald TranscriptApr 19 2017, 9:21 PM

epriestley accepted this revision.Apr 19 2017, 9:21 PM

This revision is now accepted and ready to land.Apr 19 2017, 9:21 PM

Harbormaster completed remote builds in B16595: Diff 42660.Apr 19 2017, 9:22 PM

Closed by commit rPdb60af7ea57c: Set all room settings at once (authored by chad, committed by chad). · Explain WhyApr 19 2017, 9:39 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceUpdateController.php

6 lines

Diff 42661

View Options

src/applications/conpherence/controller/ConpherenceUpdateController.php

Set all room settings at onceClosedPublicActions