Age | Commit message (Collapse) | Author | |
---|---|---|---|
2017-10-12 | Merge branch '37691-subscription-fires-multiple-notifications' into 'master' | Sean McGivern | |
fix multiple notifications from being sent for multiple labels Closes #37691 See merge request gitlab-org/gitlab-ce!14798 | |||
2017-10-10 | fix multiple notifications from being sent for multiple labels | micael.bergeron | |
This also refactor the email_helper support spec to watch for multiple emails being sent. | |||
2017-09-23 | Send a confirmation email when the user adds a secondary email address. ↵ | Brett Walker | |
Utilizes the Devise `confirmable` capabilities. Issue #37385 | |||
2017-08-14 | Whitelist or fix additional `Gitlab/PublicSend` cop violations | Robert Speicher | |
An upcoming update to rubocop-gitlab-security added additional violations. | |||
2017-08-12 | skip the :read_project check for new_project_member | http://jneen.net/ | |
since we're just adding them as a member, the permission may still return false. | |||
2017-08-12 | check notifiability for more emails | http://jneen.net/ | |
2017-08-03 | another rubocop style fix | http://jneen.net/ | |
2017-08-03 | remove build_relabeled_recipients | http://jneen.net/ | |
2017-08-03 | move the read_ability logic into NotificationRecipient | http://jneen.net/ | |
2017-08-03 | deparameterize `project` | http://jneen.net/ | |
since 99% of the time it's `target.project` anyways. | |||
2017-08-03 | default the project to target.project | http://jneen.net/ | |
2017-08-03 | require that the user be able to :read_build | http://jneen.net/ | |
to get a pipeline_failed email | |||
2017-08-03 | make sure users have to be able to read_pipeline | http://jneen.net/ | |
to get pipeline failed notifications | |||
2017-08-03 | move the #build_* methods to static, parameterize the project | http://jneen.net/ | |
2017-08-03 | factor out .notifiable_users | http://jneen.net/ | |
2017-08-03 | rm the @builder argument and factor out .notifiable_users | http://jneen.net/ | |
2017-07-27 | notification email on add new gpg key | Alexis Reigel | |
2017-06-15 | Deserialise existing custom notification settings | Sean McGivern | |
Create a post-deployment migration to update all existing notification settings with at least one custom level enabled to the new format. Also handle the same conversion when updating settings, to catch any stragglers. | |||
2017-05-10 | Enable the Style/TrailingCommaInArguments cop | Rémy Coutable | |
Use the EnforcedStyleForMultiline: no_comma option. Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-05-04 | [Multiple issue assignees] Resolving conflicts | Valery Sizov | |
2017-05-04 | Backport of multiple_assignees_feature [ci skip] | Valery Sizov | |
2017-04-03 | Quiet pipeline emails | Sean McGivern | |
1. Never send a pipeline email to anyone other than the user who created the pipeline. 2. Only send pipeline success emails to people with the custom notification setting for enabled. Watchers and participants will never receive this. 3. When custom settings are unset (for new settings and legacy ones), act as if failed_pipeline is set. | |||
2017-03-18 | Merge branch 'master' into sh-bring-back-option-to-be-notified-of-own-activity | Stan Hu | |
2017-03-17 | Resolve "Extract logic of who should receive notification into separate classes" | Dongqing Hu | |
2017-03-17 | Revert "Merge branch '8836-mr-revert' into 'master' | Stan Hu | |
This reverts commit 68e40bd49fde7b790bb31b9ac85a249bedd817d2, reversing changes made to 2d1f823b4c8b60cee525384cb52e547d2be8925a. | |||
2017-03-15 | Revert "Merge branch 'option-to-be-notified-of-own-activity' into 'master' | Stan Hu | |
This reverts commit 5e9666880376b3f53edb95cba77b5642d3cc1810, reversing changes made to b35378a938e22f745b6c6ea32b53cb50f9b6c627. | |||
2017-03-09 | use policies to protect sending email | http://jneen.net/ | |
2017-03-01 | Update occurrences of MWBS to MWPS | James Lopez | |
Rename column in the database Rename fields related to import/export feature Rename API endpoints Rename documentation links Rename the rest of occurrences in the code Replace the images that contain the words "build succeeds" and docs referencing to them Make sure pipeline is green and nothing is missing. updated doc images renamed only_allow_merge_if_build_succeeds in projects and fixed references more updates fix some spec failures fix rubocop offences fix v3 api spec fix MR specs fixed issues with partials fix MR spec fix alignment add missing v3 to v4 doc wip - refactor v3 endpoints fix specs fix a few typos fix project specs copy entities fully to V3 fix entity error more fixes fix failing specs fixed missing entities in V3 API remove comment updated code based on feedback typo fix spec | |||
2017-02-04 | Remove `try` from NotificationService#build_recipients | Richard Macklin | |
After refactoring pipeline_finished to avoid passing `nil` for current_user, we shouldn't need to use `try` here anymore. | |||
2017-02-04 | Refactor NotificationService#pipeline_finished to use skip_current_user | Richard Macklin | |
instead of passing nil for current_user | |||
2017-02-01 | Update NotificationService to respect User#notified_of_own_activity | Richard Macklin | |
2017-01-30 | Fix notification when global=disabled, group=watch | amaia | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-01-25 | address comments | Jarka Kadlecova | |
2017-01-19 | Support notes without project | Jarka Kadlecova | |
2017-01-05 | Introduce EXCLUDED_WATCHER_EVENTS to exclude events | Lin Jen-Shin | |
for watchers. Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8176#note_20920905 | |||
2017-01-05 | Make successful pipeline emails off for watchers | Lin Jen-Shin | |
Closes #24845 | |||
2016-11-24 | Rephrase some system notes to be compatible with new system note style | Douwe Maan | |
2016-11-17 | Refactoring notification service to find subscriptions per project | Douglas Barbosa Alexandre | |
2016-11-17 | Fix small typo on NotificationService | Douglas Barbosa Alexandre | |
2016-10-21 | Well pusher's privilege for read_build could be removed | Lin Jen-Shin | |
2016-10-21 | Send only to users have :read_build access, feedback: | Lin Jen-Shin | |
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_17193335 | |||
2016-10-20 | Should send to notification_email instead, feedback: | Lin Jen-Shin | |
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_17146325 | |||
2016-10-18 | Use bcc for pipeline emails because: | Lin Jen-Shin | |
We use bcc here because we don't want to generate this emails for a thousand times. This could be potentially expensive in a loop, and recipients would contain all project watchers so it could be a lot. | |||
2016-10-17 | recipients should be a list of emails | Lin Jen-Shin | |
2016-10-17 | Revert "Revert this until we do notifications" | Lin Jen-Shin | |
This reverts commit c5e0305d5fe150acd74a66efa46e3ee741642978. | |||
2016-10-17 | Revert "Split notification integration into another branch" | Lin Jen-Shin | |
This reverts commit 1404aa8677969a03ed56e8f8350257f317f576d8. | |||
2016-10-11 | Make guests unable to view MRs | Valery Sizov | |
2016-10-02 | Fix RuboCop failure in app/services/notification_service.rb | Robert Speicher | |
2016-09-29 | Notify current_user about automatic merge after successful build | Dimitris Karakasilis | |
Fixes: https://gitlab.com/gitlab-org/gitlab-ce/issues/14409 | |||
2016-08-19 | Fix notification_service argument error of declined invitation emails | Paco Guzman | |
Removing unused access_level parameter. Bug introduced here -> 6d103a2f4764441b1650ba6d790732056c9a8516 access_level parameter wasn’t needed since -> 88c2639356e6e4b4b05d96584768e07a9ce4b048 |