Skip to content
Snippets Groups Projects

Handle updates better in guest import

Merged Andreas Ellewsen requested to merge adjust-guest-import-script into master

Introduces the --update-existing option, set to False by default. Thie option controls whether or not we want to update already imported information with the info from the file.

This is usually not what we want since sponsors may have changed the end dates for already imported roles after the previous import. We choose to leave it as an option so that we CAN overwrite information if we really want.

This commit also removes the ignore-id-req option since that only makes matter worse than not importing the person.

Merge request reports

Pipeline #169420 passed

Pipeline passed for ca3d5caa on adjust-guest-import-script

Test coverage 61.86% (2.00%) from 2 jobs
Approved by

Merged by Andreas EllewsenAndreas Ellewsen 2 years ago (Dec 5, 2022 9:58am UTC)

Merge details

  • Changes merged into master with e38c6bb4.
  • Deleted the source branch.

Pipeline #170088 passed

Pipeline passed for e38c6bb4 on master

Test coverage 61.86% (2.00%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading