Software Publico / Gitlab

10 Apr, 2014

1 commit

89ecb191 Merge branch 'ensure_default_fields' into 'master' ... Browse Dir »
```
Add values to default fields to prevent 500 error when migrate from MySQL to Postgres
```
Dmitriy Zaporozhets
2014-04-10 07:20:45 +0000

09 Apr, 2014

2 commits

36f861f1 Re-annotate models ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-09 15:05:03 +0300
1bd28994 Use ProjectWiki instead of GollumWiki in code ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-09 14:35:58 +0300

08 Apr, 2014

1 commit

684877ac Fix test broken by 89c942908cc1d98dfd4af1f89b69704f691a0edf ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-08 15:58:28 +0300

07 Apr, 2014

1 commit

021faad3 Add nickname to oauth_spec ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-07 16:03:05 +0300

06 Apr, 2014

1 commit

6a9eb4b2 Improved Slack integration with message attachments. Browse Dir »

Federico Ravasio
2014-04-06 11:57:48 +0200

04 Apr, 2014

1 commit

14b68586 Merge branch 'no_email_on_notification_level_change' into 'master' ... Browse Dir »
```
Don't send emails on notification level change
```
Dmitriy Zaporozhets
2014-04-04 09:30:16 +0000

03 Apr, 2014

5 commits

860f159d Add note when MR assignee changes. Add more tests to MR services ... Browse Dir »
```
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
```
Dmitriy Zaporozhets
2014-04-03 18:16:05 +0300
eca269f7 Fix user_project observer test. Browse Dir »

Marin Jankovski
2014-04-03 16:35:33 +0200
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

2 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