gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.

Age	Commit message (Collapse)	Author
2018-07-27	Enable frozen string in app/models/*.rb	gfyoung
	Partially addresses #47424.
2018-06-21	Changes tab VUE refactoring	Felipe Artur

2017-12-07	Allow commenting on individual commits inside an MR	Douwe Maan

2017-10-07	Commenting on image diffs	Felipe Artur

2017-07-21	Add full JSON endpoints for issue notes and discussions	Douwe Maan

2017-06-01	Fix replying to a commit discussion displayed in the context of an MR	Douwe Maan

2017-05-31	Move includes call to scope	Douwe Maan

2017-05-30	Resolve N+1 query issue with discussions	Douwe Maan

2017-04-07	Fix bug where commit comment would not show up in the right discussion on ↵	Douwe Maan
	the MR page
2017-04-06	Refactor resolvability checks based on type	Douwe Maan

2017-04-05	Add line breaks in long comments	Douwe Maan

2017-04-05	Address review comments	Douwe Maan

2017-04-05	Address review comments	Douwe Maan

2017-04-05	Don't use original_discussion_id	Douwe Maan

2017-04-05	Extract commonalities between DiffDiscussion and LegacyDiffDiscussion	Douwe Maan

2017-04-05	Add specs	Douwe Maan

2017-04-05	Better notification emails for notes and (diff) discussions	Douwe Maan

2017-04-05	Add option to start a new discussion on an MR	Douwe Maan

2016-12-05	Feature: delegate all open discussions to Issue	Bob Van Landuyt
	When a merge request can only be merged when all discussions are resolved. This feature allows to easily delegate those discussions to a new issue, while marking them as resolved in the merge request. The user is presented with a new issue, prepared with mentions of all unresolved discussions, including the first unresolved note of the discussion, time and link to the note. When the issue is created, the discussions in the merge request will get a system note directing the user to the newly created issue.
2016-11-28	Tidy up text emails	Sean McGivern

2016-11-25	Add keyword arguments to truncated_diff method	hhoopes
	* Added keyword arguments to truncated_diff_lines method to allow for using highlighting or not (html templates vs. text) * Tweaked templates for consistency and format appropriateness
2016-11-25	Change diff highlight/truncate for reusability	hhoopes
	Previously the `truncated_diff_lines` method for outputting a discussion diff took in already highlighted lines, which meant it wasn't reuseable for truncating ANY lines. In the way it was used, it also meant that for any email truncation, the whole diff was being highlighted before being truncated, meaning wasted time highlighting lines that wouldn't even be used (granted, they were being memoized, so perhaps this wasn't that great of an issue). I refactored truncation away from highlighting, in order to truncate formatted diffs for text templates in email, using `>`s to designate each line, but otherwise retaining the parsing already done to create `diff_lines`. Additionally, while notes on merge requests or commits had already been tested, there was no existing test for notes on a diff on an MR or commit. Added mailer tests for such, and a unit test for truncating diff lines.
2016-09-06	Optimize discussion notes resolving and unresolving	Alejandro Rodríguez
	Use `update_all` to only require one query per discussion to update the notes resolved status. Some changes had to be made to the discussion spec to accout for the fact that notes are not individually updated now
2016-08-19	Fix bug where notes weren’t shown in discussion when the code had changed ↵	Douwe Maan
	after creation
2016-08-17	Store discussion_id on Note for faster discussion lookup.	Douwe Maan

2016-08-16	Address review feedback	Douwe Maan

2016-08-12	Add specs for Discussion and MergeRequest models	Douwe Maan

2016-08-04	Merge branch 'master' into diff-line-comment-vuejs	Douwe Maan
	# Conflicts: # app/models/discussion.rb # db/schema.rb
2016-08-02	Speedup DiffNote#active? on discussions, preloading noteables and avoid ↵	Paco Guzman
	touching git repository to return diff_refs when possible - Preloading noteable we share the same noteable instance when more than one discussion refers to the same noteable. - Any other call to that object that is cached in that object will be for any discussion. - In those cases where merge_request_diff has all the sha stored to build a diff_refs get that diff_refs using directly those sha instead accessing to the git repository to first get the commits and later the sha.
2016-07-29	Use correct user for 'Discussion resolved by...'	Douwe Maan

2016-07-27	Fixed close merge request not hiding	Phil Hughes

2016-07-27	Fix Discussion#resolved? to be false for commit discussions	Douwe Maan

2016-07-26	Add resolved and last updated status to collapsed discussions	Douwe Maan

2016-07-26	Collapse/hide resolved discussions	Douwe Maan

2016-07-26	Add access checks for diff note and discussion resolution	Douwe Maan

2016-07-26	Add methods to resolve diff notes and discussions	Douwe Maan

2016-07-26	Add resolved methods on Discussion	Douwe Maan

2016-07-21	Add Discussion model to represent MR/diff discussion	Douwe Maan