summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update time format of duration18181-pipeline-durationAnnabel Dunstone2016-07-062-1/+8
|
* Merge branch 'escape-to-cancel-note' into 'master' Jacob Schatz2016-07-061-9/+39
|\ | | | | | | | | | | | | Cancel creating or editing note by hitting Escape /cc @rspeicher See merge request !5075
| * Use switch/when statementescape-to-cancel-noteDouwe Maan2016-07-051-21/+24
| |
| * Double to single quotesDouwe Maan2016-07-041-1/+1
| |
| * Add confirmation when canceling creating/editing with changesDouwe Maan2016-07-041-0/+12
| |
| * Cancel creating or editing note by hitting EscapeDouwe Maan2016-07-041-5/+20
| |
* | Merge branch 'remove-lowercase-gitlab-ci-yml' into 'master' Robert Speicher2016-07-0615-238/+3
|\ \ | | | | | | | | | | | | Remove duplicate templates that are lowercase See merge request !5114
| * | Fix specsRémy Coutable2016-07-061-3/+3
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | Remove duplicate templates that are lowercaseRémy Coutable2016-07-0614-235/+0
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch '15094-throttling-pushes_since_gc' into 'master' Douwe Maan2016-07-062-2/+9
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Throttle the update of `project.pushes_since_gc` to 1 minute ## What does this MR do? Throttle the update of `project.pushes_since_gc` to alleviate DB load ## What are the relevant issue numbers? Relates to #15094 ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - ~~[ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)~~ - ~~[ ] API support added~~ - Tests - ~~[ ] Added for this feature/bug~~ - [x] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5083
| * | | Throttle the update of `project.pushes_since_gc` to 1 minute15094-throttling-pushes_since_gcPaco Guzman2016-07-052-2/+9
| | | |
* | | | Merge branch '19092-fix-event-for-legacydiffnote-not-considered-note' into ↵Douwe Maan2016-07-063-3/+34
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Fix diff comments not showing up in activity feed ## What does this MR do? It fixes the detection of note events to check for `Note` and `LegacyDiffNote`. ## Are there points in the code the reviewer needs to double check? No? /cc @DouweM (since I believe you introduced `LegacyDiffNote` ## Why was this MR needed? To fix #19092. ## What are the relevant issue numbers? Fixes #19092. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [x] Added for this feature/bug - [ ] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5069
| * | | | Use a more future-proof check for Note/LegacyDiffNote19092-fix-event-for-legacydiffnote-not-considered-noteRémy Coutable2016-07-062-5/+5
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | Fix diff comments not showing up in activity feedRémy Coutable2016-07-043-3/+34
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | Merge branch 'shell-version-bump' into 'master' Douwe Maan2016-07-061-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bump GITLAB_SHELL_VERSION to 3.2.0 Bump GITLAB_SHELL_VERSION to 3.2.0 Needed for !4696 to properly work. Blocked by gitlab-org/gitlab-shell!62 See merge request !5101
| * | | | | Bump GITLAB_SHELL_VERSION to 3.2.0shell-version-bumpPatricio Cano2016-07-051-1/+1
| | | | | |
* | | | | | Merge branch 'gh-labels' into 'master' Robert Speicher2016-07-0611-54/+123
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow `?`, or `&` for label names Closes #18727 See merge request !4724
| * | | | | | Fix unescaped strings in the labels dropdown templategh-labelsDouglas Barbosa Alexandre2016-07-061-2/+2
| | | | | | |
| * | | | | | Update CHANGELOGDouglas Barbosa Alexandre2016-07-051-0/+1
| | | | | | |
| * | | | | | Render label name contains ?, & in the labels dropdown without escapingDouglas Barbosa Alexandre2016-07-052-6/+5
| | | | | | |
| * | | | | | Use CGI.unescapeHTML rather than doing the gsub with a mapDouglas Barbosa Alexandre2016-07-052-12/+2
| | | | | | |
| * | | | | | Move `unescape_html_entities` from LabelsHelper to Label modelDouglas Barbosa Alexandre2016-07-054-19/+13
| | | | | | |
| * | | | | | Render references for labels that name contains ?, or &Douglas Barbosa Alexandre2016-07-056-17/+77
| | | | | | |
| * | | | | | Allow '?', or '&' for label titlesDouglas Barbosa Alexandre2016-07-056-44/+69
| |/ / / / /
* | | | | | Merge branch 'fix/import-url-validator' into 'master' Robert Speicher2016-07-066-4/+156
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixing URL validation for import_url on projects Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/17536 This MR fixes problems related to bypassing `import_url` validation on projects. This makes sure the URL is properly validated so we don't enter crap and fail while running workers that handle this URL. It also adds a migration to fix current invalid `import_url`s See merge request !4753
| * | | | | refactor url validator to use sanitizer for checkfix/import-url-validatorJames Lopez2016-07-011-3/+1
| | | | | |
| * | | | | typoJames Lopez2016-06-301-1/+1
| | | | | |
| * | | | | few more changes from suggestionsJames Lopez2016-06-304-5/+4
| | | | | |
| * | | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-06-30414-768/+1863
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix/import-url-validator # Conflicts: # spec/models/project_spec.rb
| * | | | | | use class methodJames Lopez2016-06-301-7/+8
| | | | | | |
| * | | | | | few changes based on feedbackJames Lopez2016-06-305-17/+20
| | | | | | |
| * | | | | | fix merge conflictsJames Lopez2016-06-28279-1673/+4090
| |\ \ \ \ \ \
| * | | | | | | added more info on how addressable URI differs from what we use in UrlValidatorJames Lopez2016-06-241-1/+3
| | | | | | | |
| * | | | | | | updated validator based on feedbackJames Lopez2016-06-231-3/+3
| | | | | | | |
| * | | | | | | fix chnagelogJames Lopez2016-06-221-2/+3
| | | | | | | |
| * | | | | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-06-221-2/+5
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | fix/import-url-validator
| * | | | | | | | another fix and fixed specJames Lopez2016-06-222-1/+10
| | | | | | | | |
| * | | | | | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-06-22324-1798/+3805
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix/import-url-validator
| * | | | | | | | | fix commentJames Lopez2016-06-201-1/+1
| | | | | | | | | |
| * | | | | | | | | fixed a few MySQL issues and added changelogJames Lopez2016-06-203-14/+46
| | | | | | | | | |
| * | | | | | | | | started working on a migration for projects that have current import_url issuesJames Lopez2016-06-202-7/+59
| | | | | | | | | |
| * | | | | | | | | fix addressable url validatorJames Lopez2016-06-201-7/+7
| | | | | | | | | |
| * | | | | | | | | Merge branches 'fix/import-url-validator' and 'master' of ↵James Lopez2016-06-20268-1071/+11528
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gitlab.com:gitlab-org/gitlab-ce into fix/import-url-validator
| * | | | | | | | | | fixing URL validation for import_url on projectsJames Lopez2016-06-173-3/+55
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'update-templates-for-8-10' into 'master' Rémy Coutable2016-07-0625-24/+526
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add and update .gitignore & .gitlab-ci.yml templates for 8.10 See merge request !5107
| * | | | | | | | | | | Add and update .gitignore & .gitlab-ci.yml templates for 8.10update-templates-for-8-10Rémy Coutable2016-07-0625-24/+526
| | |_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | | | | Merge branch 'fix/project-export-error' into 'master' Rémy Coutable2016-07-064-3/+9
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add more debug info to import/export and memory killer This should help debug https://gitlab.com/gitlab-org/gitlab-ce/issues/19124 further See merge request !5108
| * | | | | | | | | | fix typofix/project-export-errorJames Lopez2016-07-061-1/+1
| | | | | | | | | | |
| * | | | | | | | | | even more debugJames Lopez2016-07-063-3/+5
| | | | | | | | | | |
| * | | | | | | | | | add more debug info to project exportJames Lopez2016-07-051-0/+4
| | | | | | | | | | |