Conpherence - fix add participant / remove participant form
ClosedPublic
Actions

Authored by btrahan on Dec 9 2014, 6:46 PM.

Details

Reviewers

epriestley
chad

Maniphest Tasks

T6713: Adding a comment to a conpherence sometimes duplicates the message in the UI

Commits

Restricted Diffusion Commit
rPd5e7cd5590e9: Conpherence - fix add participant / remove participant form

Summary

Ref T6713. We were dropping latest transaction ID. This should fix the "easy" part of T6713.

Test Plan

tried to add participants and it worked! (removing participants only allows yourself, but that worked too.)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 26300.Dec 9 2014, 6:46 PM

btrahan added a task: T6713: Adding a comment to a conpherence sometimes duplicates the message in the UI.

btrahan retitled this revision from to Conpherence - fix add participant / remove participant form.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added reviewers: epriestley, chad.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptDec 9 2014, 6:46 PM

epriestley accepted this revision.Dec 9 2014, 7:11 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 9 2014, 7:11 PM

Closed by commit rPd5e7cd5590e9: Conpherence - fix add participant / remove participant form (authored by btrahan, committed by btrahan). · Explain WhyDec 9 2014, 7:29 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceUpdateController.php

12 lines

Diff 26302

View Options

src/applications/conpherence/controller/ConpherenceUpdateController.php

Conpherence - fix add participant / remove participant formClosedPublicActions