Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix unexpected extra notification mails | Patrick Derichs | 2019-04-17 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using custom_action and recipient filtering Add more generic filtering to user_ids_notifiable_on Add changelog entry Remove commented class Method is no longer needed Overloading no longer required Filter by action just in case of custom notification level Add participant check Fix unexpected extra notification mails Using custom_action and recipient filtering Add more generic filtering to user_ids_notifiable_on Add changelog entry Remove commented class Method is no longer needed Overloading no longer required Filter by action just in case of custom notification level Fix comment Add repond_to? checks Reverted custom_action filtering Enhanced output of should_email helper Changed :watch to :participating for custom notifiable users Change spec variable name Enhanced participating check These conditions are no longer needed Fix custom notification handling for participating type Participating level should include maintainers Fixed add_guest notification Fix successful pipeline notification Refactoring: Use maintainer? method on team instead Add spec for new_issue: true for a custom group setting which should have lower prio than an available project setting Clean up specs | ||||
* | [CE] Support multiple assignees for merge requestsosw-multi-assignees-merge-requests | Oswaldo Ferreira | 2019-04-08 | 1 | -17/+7 |
| | | | | | Backports https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/10161 (code out of ee/ folder). | ||||
* | Reduce remaining diff with EE in app/services | Rémy Coutable | 2019-02-14 | 1 | -1/+6 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Send a notification email on mirror update errors | Alejandro Rodríguez | 2018-12-11 | 1 | -0/+23 |
| | | | | | | The email is sent to project maintainers containing the last mirror update error. This will allow maintainers to set alarms and react accordingly. | ||||
* | Disable existing offenses for the CodeReuse cops | Yorick Peterse | 2018-09-11 | 1 | -0/+18 |
| | | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop. | ||||
* | Allow extensible mention type action for EE | Mark Chao | 2018-08-14 | 1 | -1/+7 |
| | |||||
* | Allow NotificationRecipientService::Builder::Default to handle target ↵ | Mark Chao | 2018-08-14 | 1 | -1/+1 |
| | | | | without project (e.g. Epic) | ||||
* | Add email_events to replace EMAIL_EVENTS because it needs to be dynamic, | Mark Chao | 2018-08-14 | 1 | -1/+1 |
| | | | allowing override for EE. | ||||
* | Enable frozen string in apps/uploaders/*.rb | gfyoung | 2018-07-16 | 1 | -0/+2 |
| | | | | Partially addresses #47424. | ||||
* | Resolve Naming/UncommunicativeMethod | Lin Jen-Shin | 2018-07-09 | 1 | -6/+6 |
| | |||||
* | Use stable gitlab-styles and eliminate offenses | Lin Jen-Shin | 2018-07-09 | 1 | -0/+1 |
| | |||||
* | Updates from `rubocop -a` | Lin Jen-Shin | 2018-07-09 | 1 | -2/+0 |
| | |||||
* | Notify with email when merge request became unmergeable | lulalala | 2018-05-17 | 1 | -0/+25 |
| | | | | Display MR unmergeable reasons | ||||
* | Backport 5480-epic-notifications from EE | Mario de la Ossa | 2018-05-07 | 1 | -5/+30 |
| | |||||
* | Merge branch 'master' into 'stuartnelson3/gitlab-ce-stn/issue-due-email' | Sean McGivern | 2018-04-10 | 1 | -2/+1 |
|\ | | | | | | | # Conflicts: # db/schema.rb | ||||
| * | Refactor `add_recipients`blackst0ne-rails5-fix-frozen-array | blackst0ne | 2018-04-09 | 1 | -3/+2 |
| | | |||||
| * | [Rails5] Fix spec/requests/projects/cycle_analytics_events_spec.rb | blackst0ne | 2018-04-07 | 1 | -1/+1 |
| | | |||||
* | | Only send issue due emails to participants and custom subscribers | Sean McGivern | 2018-03-30 | 1 | -1/+2 |
| | | |||||
* | | Send issue due emails to all participants | Sean McGivern | 2018-03-30 | 1 | -2/+2 |
| | | |||||
* | | Send email to recipients | Stuart Nelson | 2018-03-26 | 1 | -2/+8 |
|/ | |||||
* | Add discussion APIjprovazn-api | Jan Provaznik | 2018-03-07 | 1 | -1/+1 |
| | | | | | * adds basic discussions API for issues and snippets * reorganizes notes specs (so same tests can be used for all noteable types - issues, MRs, snippets) | ||||
* | Initial work to add notification reason to emails | Mario de la Ossa | 2018-01-16 | 1 | -22/+26 |
| | | | | | | | | | | | Adds `#build_notification_recipients` to `NotificationRecipientService` that returns the `NotificationRecipient` objects in order to be able to access the new attribute `reason`. This new attribute is used in the different notifier methods in order to add the reason as a header: `X-GitLab-NotificationReason`. Only the reason with the most priority gets sent. | ||||
* | Fix watch level for mentions in description | Sean McGivern | 2017-12-04 | 1 | -1/+12 |
| | | | | | | | | | | | | | | | For a user with the mention notification level set, the type of their corresponding NotificationRecipient must be :mention for them to receive an email. We set this correctly on notes, but we weren't adding it on new issues or MRs - perhaps because these users are also participants. But the type of the NotificationRecipient in that case would be :participant, not mention, so we have to add the mentioned users manually when creating an issue or MR. When editing an issue or MR, and there are newly-mentioned users to email, we still use the :new_issue and :new_merge_request actions, so this works for that case as well. | ||||
* | actually use the :participating notification type | http://jneen.net/ | 2017-08-21 | 1 | -1/+1 |
| | |||||
* | rubocop style fix | http://jneen.net/ | 2017-08-03 | 1 | -1/+1 |
| | |||||
* | add a comment warning that note.project may be nil | http://jneen.net/ | 2017-08-03 | 1 | -0/+4 |
| | |||||
* | remove build_relabeled_recipients | http://jneen.net/ | 2017-08-03 | 1 | -19/+0 |
| | |||||
* | move the read_ability logic into NotificationRecipient | http://jneen.net/ | 2017-08-03 | 1 | -16/+0 |
| | |||||
* | rm unused methods | http://jneen.net/ | 2017-08-03 | 1 | -26/+0 |
| | |||||
* | disable the delegate cop | http://jneen.net/ | 2017-08-03 | 1 | -0/+1 |
| | |||||
* | deparameterize `project` | http://jneen.net/ | 2017-08-03 | 1 | -13/+17 |
| | | | | since 99% of the time it's `target.project` anyways. | ||||
* | unmemoize read_ability | http://jneen.net/ | 2017-08-03 | 1 | -8/+9 |
| | | | | since it's only called once now in make_recipient | ||||
* | rm unused NewNote#subject | http://jneen.net/ | 2017-08-03 | 1 | -4/+0 |
| | | | | | its functionality is swept into the project permission check in NotificationRecipient#has_access? now | ||||
* | clearer argument name | http://jneen.net/ | 2017-08-03 | 1 | -2/+2 |
| | |||||
* | short-circuit if there is no policy, and add :read_project check | http://jneen.net/ | 2017-08-03 | 1 | -5/+1 |
| | |||||
* | force queries to include notification settings | http://jneen.net/ | 2017-08-03 | 1 | -2/+11 |
| | |||||
* | use a simple pluck, since equivalent filtering happens later | http://jneen.net/ | 2017-08-03 | 1 | -3/+1 |
| | |||||
* | style fixes | http://jneen.net/ | 2017-08-03 | 1 | -1/+2 |
| | |||||
* | .notifiable_users: compact the passed-in users | http://jneen.net/ | 2017-08-03 | 1 | -1/+1 |
| | |||||
* | default the project to target.project | http://jneen.net/ | 2017-08-03 | 1 | -1/+2 |
| | |||||
* | use intersection and diff operators instead of each | http://jneen.net/ | 2017-08-03 | 1 | -22/+2 |
| | |||||
* | make sure #custom_action is always defined | http://jneen.net/ | 2017-08-03 | 1 | -0/+4 |
| | |||||
* | move Recipient to its own NotificationRecipient file | http://jneen.net/ | 2017-08-03 | 1 | -109/+7 |
| | |||||
* | move the #build_* methods to static, parameterize the project | http://jneen.net/ | 2017-08-03 | 1 | -31/+26 |
| | |||||
* | factor out .notifiable_users | http://jneen.net/ | 2017-08-03 | 1 | -36/+4 |
| | |||||
* | don't elevate to :watch if no @custom_action is provided | http://jneen.net/ | 2017-08-03 | 1 | -1/+1 |
| | |||||
* | rm the @builder argument and factor out .notifiable_users | http://jneen.net/ | 2017-08-03 | 1 | -13/+37 |
| | |||||
* | move filtering logic into Recipient class | http://jneen.net/ | 2017-08-03 | 1 | -99/+102 |
| | |||||
* | add builder helpers with levels and start to separate out #filter! | http://jneen.net/ | 2017-08-03 | 1 | -54/+113 |
| | |||||
* | make recipients mutative during the build | http://jneen.net/ | 2017-08-03 | 1 | -64/+73 |
| |