Page MenuHomePhabricator

Remove "participationStatus" from ConpherenceParticipant
ClosedPublic

Authored by epriestley on Apr 19 2017, 7:23 PM.
Tags
None
Referenced Files
F14025652: D17730.diff
Thu, Nov 7, 7:22 PM
F14025045: D17730.diff
Thu, Nov 7, 2:10 PM
F14024857: D17730.diff
Thu, Nov 7, 12:49 PM
F14012746: D17730.diff
Fri, Nov 1, 5:29 PM
F13986661: D17730.diff
Mon, Oct 21, 4:39 AM
F13974513: D17730.diff
Oct 18 2024, 5:57 AM
Unknown Object (File)
Sep 21 2024, 12:48 PM
Unknown Object (File)
Sep 16 2024, 7:41 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.