Skip to content

SourceUserMapper return real user only if accepted

Luke Duncalfe requested to merge 443533-return-reassign_to_user-only-when-accepted into master

What does this MR do and why?

In the new importer user mapping, we initially assign the imported user contributions to placeholder users rather than the real users.

Later, we ask the real user to approve the imported contributions as theirs, and we remap the imported contributions from the placeholder user to the real user.

Import::SourceUserMapper is the class responsible for returning the user record to associate with an imported user contribution. It has existing logic to return the real user if one was previously assigned to the Import::SourceUser record. But there are states within that record where the real user has not accepted the imported contributions as theirs.

This change ensures that the real user is only returned if that user has approved.

#443533 (closed)

The feature is behind a feature flag so there is no changelog.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #443533 (closed)

Edited by Luke Duncalfe

Merge request reports