Skip to content
Snippets Groups Projects

Update full-text search index when moving issues

Merged Heinrich Lee Yu requested to merge fix-full-text-search-moved-issues into master
All threads resolved!

When moving issues, we do several updates to the new issue within an outer transaction. This caused saved_changes to return only the changes of the latest update even if more changes were committed.

This meant #update_search_data! wasn't run even though we saved the title and description of the issue.

Related to #273784 (closed)

Fixing missing data will be done in a separate MR.

Edited by Michael Kozono

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • Heinrich Lee Yu resolved all threads

    resolved all threads

  • Author Maintainer

    @mkozono can you do the maintainer review please?

  • Heinrich Lee Yu requested review from @mkozono

    requested review from @mkozono

  • mentioned in issue #273784 (closed)

  • Heinrich Lee Yu added 249 commits

    added 249 commits

    Compare with previous version

  • Michael Kozono changed the description

    changed the description

  • Michael Kozono approved this merge request

    approved this merge request

  • @engwan LGTM, thank you! :thumbsup: :rocket:

  • :wave: @mkozono, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Michael Kozono enabled an automatic merge when the pipeline for ddad5662 succeeds

    enabled an automatic merge when the pipeline for ddad5662 succeeds

  • Michael Kozono mentioned in commit 95dfbd33

    mentioned in commit 95dfbd33

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading