Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FIX: Add check for existing provider_uids to generic import #26914

Merged
merged 5 commits into from
May 17, 2024
Merged

FIX: Add check for existing provider_uids to generic import #26914

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f8504b5
FIX: Add check for existing provider_uids to generic import
italoaalves May 7, 2024
d03655c
FIX: update generic script to also check for provider_name when impor…
italoaalves May 13, 2024
fc53a33
Merge remote-tracking branch 'origin/main' into fix/generic-import-us…
gschlager May 16, 2024
078a7e3
fix linting
gschlager May 16, 2024
e8044bd
small fix
gschlager May 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 4 additions & 0 deletions script/bulk_import/generic_bulk.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -623,10 +623,14 @@ def import_user_associated_accounts
SQL

existing_user_ids = UserAssociatedAccount.pluck(:user_id).to_set
existing_provider_uids = UserAssociatedAccount.pluck(:provider_uid).to_set

create_user_associated_accounts(accounts) do |row|
user_id = user_id_from_imported_id(row["user_id"])
provider_uid = row["provider_uid"]

next if user_id && existing_user_ids.include?(user_id)
next if provider_uid && existing_provider_uids.include?(provider_uid)
italoaalves marked this conversation as resolved.
Show resolved Hide resolved

{
user_id: user_id,
Expand Down