summaryrefslogtreecommitdiff
path: root/app/controllers/projects/issues_controller.rb
Commit message (Collapse)AuthorAgeFilesLines
* Backport some changes done from Time Tracking feature in EE.ee-870-backportRuben Davila2016-11-181-1/+1
|
* add "x of y tasks completed" on issuableGuilherme Salazar2016-10-281-1/+1
| | | | | | fix issues pointed out in !6527 add task completion status feature to CHANGELOG
* Use present? instead of presence on Projects::IssuesController#indexDouglas Barbosa Alexandre2016-10-191-1/+1
|
* Remove project_labels from Projects::ApplicationControllerDouglas Barbosa Alexandre2016-10-191-5/+1
|
* List group labels on project labels pageDouglas Barbosa Alexandre2016-10-191-2/+2
|
* Allow user to create a board list based on a group labelDouglas Barbosa Alexandre2016-10-191-1/+4
|
* Allow users to apply group labels on Issues/MRsDouglas Barbosa Alexandre2016-10-191-3/+6
|
* Improve issue load time performance by avoiding ORDER BY in find_by callsh-fix-issue-perf-order-by-issueStan Hu2016-10-061-1/+2
| | | | | | | | | | | The Sortable concern has a default scope that adds ORDER BY to all queries. EXPLAIN ANALYZE shows that this additional ORDER BY statement causes the SQL optimizer to use the wrong index, which leads to a load time of 2.9 s vs 0.073 ms just for the SELECT call. The minimal change here is to re-implement find_by using where and reorder to remove the ORDER BY clause in IssuesController#index. Closes #23075
* Fix IssuesController#show degradation including project on loaded notesincludes-projects-to-render-issue-notesPaco Guzman2016-09-271-1/+1
|
* Revert part of "Merge branch 'update_issues_mr_counter' into 'master'Rémy Coutable2016-09-221-2/+0
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* fix issues mr counterbarthc2016-09-201-10/+3
|
* Add bulk update support for merge requests listDmitriy Zaporozhets2016-09-081-26/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Project tools visibility levelFelipe Artur2016-09-011-1/+1
|
* Fix external issue tracker "Issues" link leading to 404sfix-external-issue-tracker-linkStan Hu2016-08-251-1/+1
| | | | | | | | a70431f874 modified the behavior to link to the external issue tracker issues URL instead of the project path URL. This restores the previous behavior. Closes #21252, #21402
* Revert the revert of Optimistic Lockingrevert_revert_issuable_lockValery Sizov2016-08-221-1/+5
|
* Merge branch 'master' into 4273-slash-commandsDouwe Maan2016-08-161-0/+2
|\ | | | | | | | | # Conflicts: # app/services/issues/create_service.rb
| * Allow `Issue` to be submitted as spamPatricio Cano2016-08-151-0/+2
| | | | | | | | | | - Added controller actions as reusable concerns - Added controller tests
* | Fixed specs and fixes based on failing specsDouwe Maan2016-08-161-7/+1
| |
* | Fix autocomplete on commit and issue/MR edit pagesDouwe Maan2016-08-121-5/+6
|/
* Remove `url_for_new_issue` helperrs-external-issue-tracker-redirectRobert Speicher2016-08-011-3/+9
| | | | | Now we link to the standard `IssuesController#new` action, and let it redirect if we're using an external tracker.
* Redirect to external issue tracker from `/issues`Robert Speicher2016-08-011-0/+7
| | | | | | | | | | | | | Prior, in order to display the correct link to "Issues" in the project navigation, we were performing a check against the project to see if it used an external issue tracker, and if so, we used that URL. This was inefficient. Now, we simply _always_ link to `namespace_project_issues_path`, and then in the controller we redirect to the external tracker if it's present. This also removes the need for the url_for_issue helper. Bonus! :tada:
* State specific default sort order for issuableszs2016-08-011-1/+2
| | | | | | | | | | | | | | | Provide more sensible default sort order for issues and merge requests based on the following table: | type | state | default sort order | |----------------|--------|--------------------| | issues | open | last created | | issues | closed | last updated | | issues | all | last created | | merge requests | open | last created | | merge requests | merged | last updated | | merge requests | closed | last updated | | merge requests | all | last created |
* Merge branch 'akismet-ui-check' into 'master' Douwe Maan2016-07-271-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Submit new issues created via the WebUI or API to Akismet for spam check on public projects. ## What does this MR do? Submit new issues created via the WebUI by non project members to Akismet for spam check. ## Why was this MR needed? Support for Akismet was added only to the API with !2266. This MR builds on that functionality to also check issues submitted via the WebUI for spam. ## What are the relevant issue numbers? Related to: - #5573 - #5932 - gitlab-com/infrastructure#14 - gitlab-com/support#61 - !2266 cc @stanhu @MrChrisW See merge request !5333
| * Refactor spam validation to a concern that can be easily reused and improve ↵akismet-ui-checkPatricio Cano2016-07-261-2/+2
| | | | | | | | legibility in `SpamCheckService`
| * Refactor `SpamCheckService` to make it cleaner and clearer.Patricio Cano2016-07-261-7/+1
| |
| * Submit all issues on public projects to Akismet if enabled.Patricio Cano2016-07-261-13/+3
| |
| * Submit new issues created via the WebUI by non project members to Akismet ↵Patricio Cano2016-07-261-1/+17
| | | | | | | | for spam check.
* | Incorporate review commentsStan Hu2016-07-261-1/+1
| |
* | Optimize maximum user access level lookup in loading of notesStan Hu2016-07-261-0/+3
|/ | | | | | | | 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.
* Merge branch 'caironoleto/gitlab-ce-cn-issue-19747'Douwe Maan2016-07-181-0/+1
|\
| * Allow bulk (un)subscription from issues in issue indexCairo Noleto2016-07-141-0/+1
| | | | | | | | fixies #19747
* | Revert "Optimistic locking for Issue and Merge Requests"revert-lock-for-issuableStan Hu2016-07-131-5/+1
|/ | | | This reverts commit c39356998b1850f3dc26fe0b987cb419c1d1afb4.
* Optimistic locking for Issue and Merge RequestsValery Sizov2016-07-121-1/+5
|
* Enable Style/EmptyLines cop, remove redundant onesrubocop/enable-cops-for-empty-linesGrzegorz Bizon2016-07-011-1/+0
|
* Add specs and improve coffescript sortable binding functionissue_15588Felipe Artur2016-06-241-1/+2
|
* Fix issue being ordered twice and callback when moving between statesFelipe Artur2016-06-241-1/+1
|
* Support for rendering/redacting multiple documentsYorick Peterse2016-06-241-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes the way certain documents are rendered (currently only Notes) and how documents are redacted. Previously both rendering and redacting would run on a per document basis. The result of this was that for every document we'd have to run countless queries just to figure out if we could display a set of links or not. This commit changes things around so that redacting Markdown documents is no longer tied into the html-pipeline Gem. This in turn allows it to redact multiple documents in a single pass, thus reducing the number of queries needed. In turn rendering issue/merge request notes has been adjusted to take advantage of this new setup. Instead of rendering Markdown somewhere deep down in a view the Markdown is rendered and redacted in the controller (taking the current user and all that into account). This has been done in such a way that the "markdown()" helper method can still be used on its own. This particular commit also paves the way for caching rendered HTML on object level. Right now there's an accessor method Note#note_html which is used for setting/getting the rendered HTML. Once we cache HTML on row level we can simply change this field to be a column and call a "save" whenever needed and we're pretty much done.
* Respond to .json onlyAlfredo Sumaran2016-06-031-1/+6
|
* WIP: allow adding and removing labels in bulkSean McGivern2016-06-031-1/+3
|
* Add label_ids to bulk_update_paramsAlfredo Sumaran2016-06-031-1/+1
|
* Ability to assign a label to multiple issuesAlfredo Sumaran2016-06-031-1/+2
|
* Merge branch 'master' into awardablesZeger-Jan van de Weg2016-05-111-11/+22
|\
| * Handle issue move access instead of raising errorfix/handle-issue-move-accessGrzegorz Bizon2016-04-301-0/+2
| | | | | | | | Closes #15533
| * Load the "New Branch" button asynchronouslyYorick Peterse2016-04-281-2/+14
| | | | | | | | | | | | | | This button depends on Issue#can_be_worked_on? which in turn depends on Issue#related_branches. By rendering this button asynchronously we can finally remove all usages of Issue#related_branches and Issue#referenced_merge_requests from the issues "show" page.
| * Fixed issue with assignee object not being returnedPhil Hughes2016-04-221-1/+1
| | | | | | | | Closes #15515
| * Merge branch 'label-text-color-sidebar' into 'master' Robert Speicher2016-04-211-1/+1
| |\ | | | | | | | | | | | | | | | | | | | | | | | | Fixes text color on labels in sidebar Previously the labels in the sidebar would just have `#FFF` text color which could cause problems with a light background color. With this, the text color comes from the JSON. See merge request !3846
| | * Fixes text color on labels in sidebarPhil Hughes2016-04-211-1/+1
| | |
| * | Merge branch 'remove-update-files' into 'master' Yorick Peterse2016-04-211-1/+0
| |\ \ | | | | | | | | | | | | | | | | Removed JS update templates See merge request !3814
| | * | Removed JS update templatesremove-update-filesPhil Hughes2016-04-191-1/+0
| | | |
| * | | Add due_date:time field to Issue modelMehmet Beydogan2016-04-201-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add due_date text field to sidebar issue#show Add ability sorting issues by due date ASC and DESC Add ability to filtering issues by No Due Date, Any Due Date, Due to tomorrow, Due in this week options Add handling issue due_date field for MergeRequest Update CHANGELOG Fix ambigous match for issues#show sidebar Fix SCREAMING_SNAKE_CASE offenses for due date contants Add specs for due date sorting and filtering on issues
View Lorry Controller Status