Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/app/helpers/notes_helper.rb
AgeCommit message (Collapse)Author
2017-09-14Support discussion locking in the backendJarka Kadlecova
2017-09-06WIP: refactor the first-contributor to Issuablemicael.bergeron
this will remove the need make N queries (per-note) at the cost of having to mark notes with an attribute this opens up the possibility for other special roles for notes
2017-09-06fix #35161, add a first-time contributor badgemicael.bergeron
a new badge will be added when an user that doesn't yet have any merged merge request is discussing on either issues or merge requests that he created. this is indented for people to use extra care when discussing with a new contributor.
2017-09-06Resolve outdated diff discussions on pushSean McGivern
2017-08-17Misc tweaksDouwe Maan
2017-08-07Convert ?full_data=1 -> ?view=full_dataSean McGivern
2017-07-28Merge branch 'fix-replying-to-commit-comment-in-mr-from-fork' into 'master'Douwe Maan
Fix replying to commit comments on merge requests created from forks Closes #31772 See merge request !13137
2017-07-28Fix replying to commit comments on MRs from forksSean McGivern
A commit comment shows in the MR, but if the MR is from a fork, it will have a different project ID to the MR's target project. In that case, add an note_project_id param so that we can pick the correct project for the note.
2017-07-25Removed inline JS in shared HAML filesPhil Hughes
2017-07-05Create and use project path helpers that only need a project, no namespaceDouwe Maan
2017-06-29Render add-diff-note button with server.Bryce Johnson
This commit moves the rendering of the button back to the server, and shows/hides it using opacity rather than display. It also removes the transform applied to the button on hover (scale). Previously, both of these factors automatically triggered a reflow, which creates a performance bottleneck on pages with larger DOM size. MR: !12103
2017-06-15Rename "Slash commands" to "Quick actions"Eric Eastwood
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/27070 Deprecate "chat commands" in favor of "slash commands" We looked for things like: - `slash commmand` - `slash_command` - `slash-command` - `SlashCommand`
2017-06-07Added more actions and report as abuse to all notesLuke "Jared" Bennett
2017-06-01Fix replying to a commit discussion displayed in the context of an MRDouwe Maan
2017-05-10Enable the Style/TrailingCommaInLiteral copRémy Coutable
Use the EnforcedStyleForMultiline: no_comma option. Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-05-05Support comments for personal snippetsJarka Kadlecova
2017-05-03Allow commenting on older versions of the diff and comparisons between diff ↵Douwe Maan
versions
2017-04-08Some code tweaksDouwe Maan
2017-04-08Fix views after rebaseDouwe Maan
2017-04-08Fix specs and make tweaksDouwe Maan
2017-04-08Add link to diff header tooDouwe Maan
2017-04-05Address review commentsDouwe Maan
2017-04-05Address review commentsDouwe Maan
2017-04-05Correctly display multiple separate discussions on the same diff lineDouwe Maan
2017-04-05Don't use original_discussion_idDouwe Maan
2017-04-05Fix some specsDouwe Maan
2017-04-05Add option to start a new discussion on an MRDouwe Maan
2016-09-20Fix note form hint showing slash commands supported for commitsRémy Coutable
Signed-off-by: Rémy Coutable <remy@rymai.me>
2016-08-17Store discussion_id on Note for faster discussion lookup.Douwe Maan
2016-08-04Comment & resolve button no longer looks for can-resolve attributePhil Hughes
Fixed some bugs when removing notes
2016-08-04Merge branch 'master' into diff-line-comment-vuejsDouwe Maan
# Conflicts: # app/models/discussion.rb # db/schema.rb
2016-08-02Speedup 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-29Moved most of the data handling into discussion & notes modelsPhil Hughes
Reduced some duplicated code with compiling components Fixed bug with resolve button tooltip not updating after resolving discussion
2016-07-29Merge branch 'master' into diff-line-comment-vuejsDouwe Maan
# Conflicts: # db/schema.rb
2016-07-27Optimize maximum user access level lookup in loading of notesStan Hu
NotesHelper#note_editable? and ProjectTeam#human_max_access currently take about 16% of the load time of an issue page. This MR preloads the maximum access level of users for all notes in issues and merge requests with several queries instead of one per user and caches the result in RequestStore.
2016-07-26Comment & resolve button text is done through VueJSPhil Hughes
2016-07-26Use sha1 of discussion ID.Douwe Maan
2016-07-24Added resolve button to discussionsPhil Hughes
Top count displays how many resolved discussions
2016-07-21Add Discussion model to represent MR/diff discussionDouwe Maan
2016-07-08Do not set `@use_legacy_diff_notes`Douwe Maan
2016-07-08Fix behavior around MRs that need legacy diff notesDouwe Maan
2016-07-08Merge branch 'master' into faster-diffsDouwe Maan
# Conflicts: # app/helpers/notes_helper.rb # app/views/projects/diffs/_line.html.haml # app/views/projects/diffs/_parallel_view.html.haml # app/views/projects/diffs/_text_file.html.haml # features/steps/shared/diff_note.rb
2016-07-07Display new diff notes and allow creation through the web interfaceDouwe Maan
2016-07-07Extract parts of LegacyDiffNote into DiffOnNote concern and move part of ↵Douwe Maan
responsibility to other classes
2016-07-07Style diff and blob file headers the same wayDouwe Maan
2016-07-06Simplify HAML code using unless condition and nestingPaco Guzman
Extract diff line data attributes to helper methods
2016-07-06Diffs will create button/diff form on demand no on server sideJacob Schatz
2016-06-29Be explicit which project and user ID are memoizedStan Hu
2016-06-29Memoize the maximum access level for the author of notesStan Hu
In #19273, we saw that retrieving ProjectTeam#human_max_access for each note takes the bulk of the time when rendering certain issues or merge requests. We observe that most of the comments in an issue are typically done by the same users. This MR memoizes the max access level by user ID.
2016-05-14Extract LegacyDiffNote out of NoteDouwe Maan
generated by cgit v1.2.3 (git 2.25.1) at 2024-08-16 16:37:38 +0300