Skip to content

Fix tenant votes minds#5004

Ben requested to merge fix/tenant-votes-m5004 into master

Ticket(s) / Related Merge Requests

minds#5004 (closed)

Summary of Changes

Metric change sockets are pulling tenant vote counts from the Cassandra "Counters" cache, when tenant votes are saved in MySQL. This was leading to discrepancies in synchronization when the runner emits a metric count change via sockets, meaning that when a user upvotes a post, the socket can send back an incorrect count, forcing the UI to update incorrectly.

Testing Considerations

Bound to https://ben.oke.minds.io/

  • I can upvote a string of content on non-tenant sandbox and the values update correctly (note: if you do this rapidly there ARE some known sync issues that this MR does not address).
  • I can upvote a string of content on a tenant sandbox and the values update correctly.

Deployment Considerations

Standard deployment to runners.

Regression Scope

Isolated to the metric change runner for votes up, and by extension live client vote updating.

Platform Affected (web, mobile, etc)

Engine.

Developer Testing Completed

Manual, unit (phpspec).

Screenshots / Screen Recording

Tenant

simplescreenrecorder-2024-07-03_14.11.38

Non-tenant

simplescreenrecorder-2024-07-03_14.16.32

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports