Skip to content

Fix ElasticSearch failures causing misapplication of hellominds tag front#5963

Ben requested to merge fix/hellominds-tag-5963 into master

Ticket(s) / Related Merge Requests

front#5963 (closed)

Summary of Changes

Fixes ElasticSearch failures causing a misapplication of hellominds tag. Current ElasticSearch is sometimes returning that a user has no posts when they infact do - this change migrates to a count query for performance and potentially better accuracy and adds better exception handling such that if there is an exception, we return that a user HAS made posts.

Testing Considerations

Log into an existing user Make a post (without the term hellominds) Search hellominds - your post should not appear

Register a new user and verify the email Make a post (without the term hellominds) Search hellominds - your post should appear

Deployment Considerations

Standard deployment to runners.

Regression Scope

Effects the showing of boost channel notice and rewards eligibility on ES error (neither should show / be possible to do in the event of an error) and of course, effects the search indexer.

Platform Affected (web, mobile, etc)

Runners

Developer Testing Completed

Unit (phpspec), manual

Screenshots / Screen Recording

image

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