Differential D17697

Clean up remove participant language in Conpherence
ClosedPublic
Actions

Authored by chad on Apr 15 2017, 3:55 AM.

Details

Reviewers

epriestley

Commits

rPa56f9a1a5599: Clean up remove participant language in Conpherence

Summary

Updates the language to use "Remove Participant" instead of "Banish User"

Test Plan

Read through the various cases, test them by removing myself or others

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Apr 15 2017, 3:55 AM

Herald added a subscriber: Korvin. · View Herald TranscriptApr 15 2017, 3:55 AM

Harbormaster completed remote builds in B16494: Diff 42561.Apr 15 2017, 3:56 AM

This is probably legitimately more clear anyway since this action is "kick" (remove temporarily; user can rejoin, not "ban" (banish permanently; user can not rejoin).

This revision is now accepted and ready to land.Apr 15 2017, 12:29 PM

Closed by commit rPa56f9a1a5599: Clean up remove participant language in Conpherence (authored by chad, committed by chad). · Explain WhyApr 15 2017, 4:07 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceUpdateController.php

8 lines

Diff 42565

View Options

src/applications/conpherence/controller/ConpherenceUpdateController.php

Clean up remove participant language in ConpherenceClosedPublicActions