Add sound to logged out Conpherence
ClosedPublic
Actions

Authored by chad on May 20 2017, 4:42 AM.

Tags

None

Referenced Files

	F18952536: D17973.id.diff
	Nov 12 2025, 6:07 AM

	F18947882: D17973.diff
	Nov 11 2025, 10:32 PM

	F18823739: D17973.id.diff
	Oct 23 2025, 1:08 PM

	F18823592: D17973.diff
	Oct 23 2025, 12:35 PM

	F18823037: D17973.diff
	Oct 23 2025, 9:55 AM

	F18783651: D17973.id43230.diff
	Oct 13 2025, 8:39 AM

	F18783384: D17973.id43230.diff
	Oct 13 2025, 6:19 AM

	F18769742: D17973.id43230.diff
	Oct 8 2025, 8:57 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T12735: InvalidArgumentException when viewing a room you haven't joined from Safari

Commits

rP1d89c1ed24ae: (stable) Add sound to logged out Conpherence
rP5f49f9c793e3: Add sound to logged out Conpherence

Summary

Fixes T12735. Adds a sound if the user is logged out, skips checking a setting.

Test Plan

set participants to null and verify sound plays, no exceptions1111

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.May 20 2017, 4:42 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 20 2017, 4:42 AM

Harbormaster completed remote builds in B17138: Diff 43230.May 20 2017, 4:43 AM

epriestley accepted this revision.May 20 2017, 11:21 AM

This revision is now accepted and ready to land.May 20 2017, 11:21 AM

Closed by commit rP5f49f9c793e3: Add sound to logged out Conpherence (authored by chad, committed by chad). · Explain WhyMay 20 2017, 1:51 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceUpdateController.php

14 lines

Diff 43240

View Options

src/applications/conpherence/controller/ConpherenceUpdateController.php

Add sound to logged out ConpherenceClosedPublicActions