Page MenuHomePhabricator

Remove "participationStatus" from ConpherenceParticipant
ClosedPublic

Authored by epriestley on Apr 19 2017, 7:23 PM.
Tags
None
Referenced Files
F19017793: D17730.id42645.diff
Sun, Nov 23, 9:18 AM
F18819595: D17730.id.diff
Oct 22 2025, 7:24 AM
F18810045: D17730.diff
Oct 19 2025, 7:03 PM
F18601265: D17730.id42645.diff
Sep 13 2025, 11:58 AM
F18089345: D17730.id42655.diff
Aug 6 2025, 11:16 AM
F18081503: D17730.id42655.diff
Aug 5 2025, 12:42 AM
F18056633: D17730.id42645.diff
Aug 4 2025, 8:42 AM
F17985352: D17730.id.diff
Aug 1 2025, 8:44 PM
Subscribers
None

Details

Summary

Pathway to D17685. This column is a very complicated cache of: is participant.messageCount equal to thread.messageCount?

We can just ask this question with a JOIN instead and simplify things dramatically.

Test Plan
  • Ran migration.
  • Browsed around.
  • Sent a message, saw unread count go up.
  • Read the message, saw unread count go down.

Diff Detail

Repository
rP Phabricator
Branch
status2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16580
Build 22104: Run Core Tests
Build 22103: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Apr 19 2017, 7:38 PM
This revision was automatically updated to reflect the committed changes.