delta/gitlab/gitlab-ce.git/app/views/projects/notes, branch koding-setting-api

Merge branch 'resolve-buttons-path' into 'master'

2016-09-26T23:00:55+00:00


Pass the full project path for resolve buttons

## What does this MR do?

The full project path is passed from the HTML into the JS rather than the resolve button JS generating the URL based on the the namespaces.

@smcgivern @stanhu Do we have anyway of adding tests for installs with relative URLs?

## What are the relevant issue numbers?

Closes #21704

See merge request !6129

Change "Please register or login" to "Please sign up or sign in"

2016-09-21T09:58:13+00:00

Now this is more consistent with the rest of the app.

Changed props to use strings

2016-09-21T09:19:47+00:00

Use jQuery

Pass the full project path for resolve buttons

2016-09-21T09:07:24+00:00

This fixes an issue that would stop the resolve buttons working when GitLab is installed with a relative URL

Closes #21704

Fix note form hint showing slash commands supported for commits

2016-09-20T11:58:14+00:00

Signed-off-by: Rémy Coutable

Only check :can_resolve permission if the note is resolvable

2016-09-12T11:47:12+00:00

Currently this permission check incurs a significant performance hit on loading issues
where it does not make sense to check this permission in the first place.

Add hover color to emoji icon

2016-08-25T17:21:37+00:00

fix location of markdown help location

2016-08-22T20:25:05+00:00

Change location of slash commands in notes widget

2016-08-22T13:02:24+00:00

Merge branch 'diff-line-comment-vuejs' into 'master'

2016-08-19T02:24:34+00:00


Diff line comments resolve

## What does this MR do?

Diff line comments can be resolved.

Part of #10325 

To do:

- [x] Backend (@DouweM)
  - [x] Fix https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5022#note_13319326. Will be made easier by https://gitlab.com/gitlab-org/gitlab-ce/issues/17237#note_13370331
  - [x] System note when all discussions are resolved
  - [x] Notification when all discussions are resolved
  - [x] Write unit tests
  - [x] Look at resolve time https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5022#note_13912743 - Fixed by 4a13aa9
- [x] Frontend (@iamphill)
  - [x] Fix bugs
  - [x] Write more feature tests 
- [x] Frontend (@connorshea)
  - [x] Address frontend feedback
  - [x] Feature specs for Jump feature
  - [x] Documentation
  - [x] Add Vue.js in a standard way

See merge request !5022