Skip to content
Snippets Groups Projects

Add epic's note system_note_meta to Import/Export

Merged George Koltsov requested to merge georgekoltsov/import-epic-system-notes into master
All threads resolved!

What does this MR do?

This MR adds system_note_metadata relation to epic's notes in order to accurately migrate system notes of an epic (e.g. when epic is related to another epic, a system note is created). This MR also:

  • Updates both Import/Export and Bulk Import codebase tests, as both features shared ndjson import/export behaviour
  • Moves epics.ndjson.gz file to ee/spec/fixtures as epics are EE feature
  • Updates ndjson fixtures to include epics with notes that have system_note_metadata

Mentions #323580 (closed)

Screenshots (strongly suggested)

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by George Koltsov

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bob Van Landuyt resolved all threads

    resolved all threads

  • Bob Van Landuyt approved this merge request

    approved this merge request

  • Bob Van Landuyt enabled an automatic merge when the pipeline for f277c2c4 succeeds

    enabled an automatic merge when the pipeline for f277c2c4 succeeds

  • Thank you @georgekoltsov. I like this kind of change :laughing: (!63551 (comment 597686107)).

  • Bob Van Landuyt mentioned in commit 4756edbe

    mentioned in commit 4756edbe

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • 🤖 GitLab Bot 🤖 added groupimport and integrate label and removed 1 deleted label

    added groupimport and integrate label and removed 1 deleted label

  • Please register or sign in to reply
    Loading