Software Publico / Gitlab

03 Apr, 2014

3 commits

9b6224f9 More test fixes ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-03 12:05:21 +0300
e1dd76d5 Fix tests that dont respect project-fork relation. pt2 ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-03 11:49:12 +0300
c31b7031 Fix tests that dont respect project-fork relation ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-03 11:47:56 +0300

02 Apr, 2014

9 commits

65cd9829 Remove MergeRequest observer ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 21:35:59 +0300
6ae25290 Remove KeysObserver ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 21:13:05 +0300
77c0a6db Remove email observer specs ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 21:03:31 +0300
3faa14e7 Merge branch 'reduce-observers' into 'master' ... Browse Dir »
```
Move issue create/update code to services

Reduce observers role in GitLab code
```
Dmitriy Zaporozhets
2014-04-02 17:54:27 +0000
49f977d6 Fix tests ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 19:55:23 +0300
928fbeee More tests for Isses services ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 16:33:07 +0300
7d8d9bd1 Remove issue observer tests ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 15:51:37 +0300
c4e81ed9 Move update issue code to separate service ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-02 13:54:41 +0300

cfd9fd30 Move code for issue creation to service. ... Browse Dir »

The goal of suych refactoring is to get rid of observers.
Its much easier to test and code when object creation and all other
related actions done in one class instead of splited across observers,
callbacks etc.

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

2014-04-02 13:38:35 +0300

01 Apr, 2014

2 commits

f4f0a7e0 Add a failing test. Browse Dir »

Marin Jankovski
2014-04-01 17:18:52 +0200
33a00cee Create branch via API ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-01 10:39:53 +0300

31 Mar, 2014

2 commits

9b276f00 Move branches api to separate class ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-03-31 16:31:53 +0300
046fa9bd Merge pull request #5469 from NARKOZ/api-pagination-headers ... Browse Dir »
```
add 'Link' header for API response
```
Dmitriy Zaporozhets
2014-03-31 13:46:57 +0300

26 Mar, 2014

5 commits

466203eb Additional check in the test. Browse Dir »

Marin Jankovski
2014-03-26 12:56:25 +0100
961d8793 Add the tests for user search. Browse Dir »

Marin Jankovski
2014-03-26 12:44:00 +0100
17fd0e5c Merge pull request #6569 from criteo/api_merge_request_comments ... Browse Dir »
```
Add method to get the comments on a merge request
```
Dmitriy Zaporozhets
2014-03-26 10:06:51 +0200

f0adf28e Merge branch 'event-create-service' into 'master' ... Browse Dir »

EventCreateService class

The goal is to collect all event creation logic in one place called EventCreateService.
Because now its placed in observers, controllers, services etc

2014-03-26 07:41:27 +0000

05c9ab94 Merge branch 'refactor-author-id-of-changes' into 'master' ... Browse Dir »
```
Remove author_id_of_changes

To prevent confusion because we already have `current_user`
```
Dmitriy Zaporozhets
2014-03-26 07:01:16 +0000

25 Mar, 2014

8 commits

81527e9f Merge pull request #6374 from zzet/patch-10 ... Browse Dir »
```
Update merge_request_observer_spec.rb
```
Dmitriy Zaporozhets
2014-03-25 19:41:47 +0200
9afccaa8 Merge pull request #6554 from tsigo/remove-spork ... Browse Dir »
```
Remove Spork
```
Dmitriy Zaporozhets
2014-03-25 19:40:50 +0200
a7aacb4d Merge pull request #6586 from abner/master ... Browse Dir »
```
added api method to return labels of a given project
```
Dmitriy Zaporozhets
2014-03-25 19:39:16 +0200
535339a6 Fix tests ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-03-25 16:25:59 +0200
45e6abff Remove activity observer specs ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-03-25 15:09:17 +0200
d89130b0 Collect all event creation logic in one place called EventCreateService ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-03-25 14:48:02 +0200
cce80d04 Remove author_id_of_changes from spec ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-03-25 14:02:55 +0200
72b13cc1 Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce Browse Dir »

Dmitriy Zaporozhets
2014-03-25 09:04:22 +0200

24 Mar, 2014

4 commits

3d8ecbc9 Merge branch 'no-emails-for-mentions' into 'master' ... Browse Dir »

 Don't send an email for "mentioned in" notes

Currently, an email is sent every time a mentionable is referenced by an issue, a commit or a merge request: if I comment "This MR is related to #5", the watchers get one email for the comment, and another one stating "Issue #5 was mentioned by issue #13".

This is annoying — but the biggest issue is when pushing an existing branch. Every issue referenced by commit messages in this branch will get a new mention (which is fine), and dozens of emails will be sent for all these new mentions (which is not).

This commit fixes the spam by avoiding to send an email when a new mention is added to an existing mentionable. In most cases the email notification sent by the mentioner is enough.

2014-03-24 18:52:41 +0000

65f5496e Fix the merge notification email not being sent ... Browse Dir »

The 'author_id_of_changes' attribute is not persisted in the database.
As we retrieve the merge request from the DB just before sending the
email, this attribute was always nil.

Also there was no tests for the merge notification code - tests have
been added.

Fix #6605

2014-03-24 15:11:35 +0100

d66d1097 Merge pull request #6594 from tsigo/rs-small-changes ... Browse Dir »
```
Small changes
```
Dmitriy Zaporozhets
2014-03-24 14:58:35 +0200
1e3b5c37 Remove Spork Browse Dir »

Robert Speicher
2014-03-24 08:55:27 -0400

22 Mar, 2014

3 commits

07833d16 Move Project's service specs to the correct location ... Browse Dir »
```
[ci skip]
```
Robert Speicher
2014-03-22 16:38:22 -0400
85b5812b moving labels specs to projects_spec file Browse Dir »

Ábner Oliveira
2014-03-22 17:29:13 -0300
760f827b Remove code that does nothing ... Browse Dir »
```
ping @karlhungus
```
Robert Speicher
2014-03-22 15:40:42 -0400

21 Mar, 2014

1 commit

44aa6b90 added api method to return labels of a given project Browse Dir »

Ábner Oliveira
2014-03-21 06:25:25 -0300

20 Mar, 2014

1 commit

fefab474 Fix `/:username.keys` response content type ... Browse Dir »

Currently this method responds with `text/html`. It is kind of unusable
if you open it in a browser. The browser thinks it is HTML and renders
it as HTML, meaning new lines are dropped. So it's very hard to
distinguish where the key starts and where it ends.

This commit changes the content type header to `text/plain`.

2014-03-20 19:09:14 +0400

19 Mar, 2014

2 commits

9886998f Add method to get the comments on a merge request ... Browse Dir »
```
Add method to get the comments for a merge request and
document that you can change the status of a merge request.
```
Stuart Pook
2014-03-19 19:07:51 +0100
492dc82c Merge pull request #6557 from tsigo/faster-merge-request-features ... Browse Dir »
```
Speed up features/notes_on_merge_requests_spec
```
Dmitriy Zaporozhets
2014-03-19 14:45:15 +0200