Skip to content
Snippets Groups Projects

campaigns: Cleanup some little things

Merged Administrator requested to merge es/syncer-experiments into main

Created by: eseliger

I found these while browsing the code, trying to understand the selection of auth tokens for the syncer.

  • The repoStore.Get method already returns an error with that error message, so no need to wrap it another time. Saves us a copy-pasted helper
  • Handle potential nil index access in buildChangesetSource
  • Correctly store error in syncChangeset so transactions are correctly rolled back

Merge request reports

Merged by avatar (Jul 8, 2025 1:08am UTC)

Loading

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