Skip to content

Conversation

@netbe
Copy link
Collaborator

@netbe netbe commented Oct 7, 2024

Issue

Note: this PR is based on #2017, so preferably start with #2017 first.

Due to some race condition, an user could end up with duplicate proteus team 1-1 conversation. This PR aims to fix this, by showing only one same on all platforms 1-1 conversation.

  • Fix one to one migration that was introduced in adding the last rule to choose the 1-1 conversation (sorted qualified id)
  • Add coredata migration post action to latest db migration to reset the 1-1 conversation if it was wrongly set before AND merge all messages into the conversation to prepare for MLS migration.
  • Remove PatchApplicator (unused)

Testing

  • Use production account where duplicate 1-1 conversation exist
  • run the migration
  • See messages have been merged into one conversation

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

@echoes-hq echoes-hq bot added the echoes: unplanned Any work item that isn’t part of the product or technical roadmap. label Oct 7, 2024
@netbe netbe force-pushed the fix/duplicate-1-1-conversations-WPB-10260 branch from 0bee456 to fd72b45 Compare October 10, 2024 15:16
@netbe netbe changed the base branch from develop to chore/cleanup-phone-number-WPB-11437 October 10, 2024 15:21
@netbe netbe marked this pull request as ready for review October 10, 2024 15:24
@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2024

Test Results

4 872 tests   4 872 ✅  11m 50s ⏱️
  562 suites      0 💤
    4 files        0 ❌

Results for commit 4948cf3.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@samwyndham samwyndham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@netbe netbe force-pushed the chore/cleanup-phone-number-WPB-11437 branch from de7da4c to d6bb78d Compare October 24, 2024 07:42
Base automatically changed from chore/cleanup-phone-number-WPB-11437 to develop October 25, 2024 15:47
netbe added 4 commits October 28, 2024 23:19
# Conflicts:
#	wire-ios-data-model/Resources/Databases Changelog.md
#	wire-ios-data-model/Source/Public/ZMUser.h
#	wire-ios-data-model/Support/Sources/ModelHelper.swift
#	wire-ios-data-model/WireDataModel.xcodeproj/project.pbxproj
…m:wireapp/wire-ios into fix/duplicate-1-1-conversations-WPB-10260

# Conflicts:
#	WireAPI/Sources/WireAPI/Network/NetworkService/NetworkServiceError.swift
#	WireAPI/Sources/WireAPI/Network/PushChannel/PushChannelServiceError.swift
#	wire-ios-build-assets
#	wire-ios-data-model/Source/ManagedObjectContext/Migration/118-119/FixDuplicateOneOnOneConversationsAction.swift
#	wire-ios-data-model/Source/Public/ZMUser.h
@netbe netbe changed the title fix: duplicate 1-1 conversations - WPB-10260 fix: duplicate 1-1 conversations - WPB-10260 Oct 29, 2024
@netbe netbe requested a review from johnxnguyen October 30, 2024 07:34
@netbe netbe enabled auto-merge October 30, 2024 07:34
@netbe netbe added this pull request to the merge queue Oct 30, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Oct 30, 2024
@netbe netbe enabled auto-merge October 30, 2024 13:03
@netbe netbe added this pull request to the merge queue Oct 30, 2024
Merged via the queue into develop with commit bfc2e94 Oct 30, 2024
12 checks passed
@netbe netbe deleted the fix/duplicate-1-1-conversations-WPB-10260 branch October 30, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: unplanned Any work item that isn’t part of the product or technical roadmap.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants