Skip to content
Snippets Groups Projects

Fixing problems with events for import/export

Merged James Lopez requested to merge fix/import-export-events into master

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19202

A couple of issues related to target being missing in exported Events (as being polymorphic and not have ActiveRecord relationships is a bit more tricky than normal models) plus as the export was in JSON, the import retrieves hashed fields as stringified hashes and not symbolized - so fixed that as well, which was the cause of https://gitlab.com/gitlab-org/gitlab-ce/issues/19202

Also fixed / refactored tests :simpl Import/Export Version has been bumped to 0.1.1 as theses changes to events won't work very well with old exports - forcing users to generate a new export in the new version.

Merge request reports

Pipeline #3629155 passed

Pipeline passed for e7319d0a on fix/import-export-events

Merged by avatar (Apr 22, 2025 3:00pm UTC)

Loading

Pipeline #3629508 failed

Pipeline failed for c368cb60 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 11 14 - :releases
    12 - :events
    13 15 - project_members:
    14 16 - :user
    15 17 - merge_requests:
    16 18 - notes:
    17 :author
    19 - :author
    20 - :events
    18 21 - :merge_request_diff
    22 - :events
    19 23 - pipelines:
    20 24 - notes:
    21 :author
    25 - :author
    26 - :events
  • James Lopez Added 129 commits:

    Added 129 commits:

    • 01f7d1e3...c578fb06 - 128 commits from branch master
    • d6d0a355 - Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into fix/import-export-events
  • James Lopez Added 1 commit:

    Added 1 commit:

    • f29c3047 - use has_many relationship with events
  • Reassigned to @rymai

  • @rymai updated!

  • James Lopez Added 1 commit:

    Added 1 commit:

    • 08b2ea51 - changelog madness
  • 30 30 - Metrics for Rouge::Plugins::Redcarpet and Rouge::Formatters::HTMLGitlab
    31 31 - Allow [ci skip] to be in any case and allow [skip ci]. !4785 (simon_w)
    32 32 - Add basic system information like memory and disk usage to the admin panel
    33 - Fixes issues importing events in Import/Export. Import/Export version bumped to 0.1.1
  • @jameslopez Please fix the CHANGELOG and squash your commits a bit and we'll be good to go!

  • James Lopez Added 1 commit:

    Added 1 commit:

    • 9bbe0332 - squashed commit - fixed import-export events
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading