Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
GitLab FOSS
GitLab FOSS
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Security & Compliance
    • Security & Compliance
    • Dependency List
  • Packages
    • Packages
    • Container Registry
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • GitLab.org
  • GitLab FOSSGitLab FOSS
  • Merge Requests
  • !7527

Merged
Opened Nov 17, 2016 by Douwe Maan@DouweM5 of 6 tasks completed5/6 tasks
  • Report abuse
Report abuse

Add option to start a new resolvable discussion in an MR

Resolves https://gitlab.com/gitlab-org/gitlab-ce/issues/24378

EE port: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1556

To do:

  • Specs
  • Reply by email support
  • Better notifications (https://gitlab.com/gitlab-org/gitlab-ce/issues/24378#note_25365790)
  • Consider the case where a new diff comment can become separated of an existing discussion (comment created while existing discussion was not visible, for example before it was loaded - uncommon, or when its position was out of date - unlikely)
  • Remove/ignore the notes.original_discussion_id column
  • Documentation https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/10390

Out of scope:

  • API support (https://gitlab.com/gitlab-org/gitlab-ce/issues/20901)

Button in form:

Screen_Shot_2016-11-17_at_13.34.37

After tabbing to the button:

Screen_Shot_2016-11-17_at_13.52.48

After hitting Command+Option+Return (or Ctrl-Alt-Enter on Windows), a new discussion appears:

Screen_Shot_2016-11-17_at_13.56.23

/cc @smcgivern

  • Discussion 229
  • Commits 87
  • Pipelines 65
  • Changes 100+
Assignee
Assign to
9.1
Milestone
9.1
Assign milestone
Time tracking
3
Labels
Plan [DEPRECATED] devops::plan merge requests
Assign labels
  • View project labels
Reference: gitlab-org/gitlab-foss!7527

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.