Skip to content

GitLab Next

    • GitLab: the DevOps platform
    • Explore GitLab
    • Install GitLab
    • How GitLab compares
    • Get started
    • GitLab docs
    • GitLab Learn
  • Pricing
  • Talk to an expert
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Menu
    Projects Groups Snippets
  • Get a free trial
  • Sign up
  • Login
  • Sign in / Register
  • GitLab FOSS GitLab FOSS
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar

GitLab 15.0 is launching on May 22! This version brings many exciting improvements, but also removes deprecated features and introduces breaking changes that may impact your workflow. To see what is being deprecated and removed, please visit Breaking changes in 15.0 and Deprecations.

  • GitLab.org
  • GitLab FOSSGitLab FOSS
  • Merge requests
  • !7527
Project 'gitlab-org/gitlab-ce' was moved to 'gitlab-org/gitlab-foss'. Please update any links and bookmarks that may still have the old path.
Merged
Created Nov 17, 2016 by Douwe Maan@DouweMContributor5 of 6 tasks completed5/6 tasks
  • Review changes

  • Download
  • Email patches
  • Plain diff

Add option to start a new resolvable discussion in an MR

  • Overview 229
  • Commits 87
  • Pipelines 64
  • Changes 100+

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

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: new-resolvable-discussion