Age | Commit message (Collapse) | Author | |
---|---|---|---|
2018-07-16 | Update notification specs after renaming a class | Grzegorz Bizon | |
2018-07-11 | Resolve "Rename the `Master` role to `Maintainer`" Backend | Mark Chao | |
2018-07-09 | This test was copied from EE | Lin Jen-Shin | |
2018-07-06 | Move spec/mailers/previews to app/mailers/previews | Lin Jen-Shin | |
2018-06-20 | Notify only when unmergeable due to conflict | Mark Chao | |
There is still the edge case when 'no commits' changes to 'conflict' would not trigger notification, which we ignore for now. Calling can_be_merged? can cause exception (e.g. non-UTF8) Ignore those by rescueing. Remove unmergeable_reason as now only conflict is notified Update spec | |||
2018-06-03 | Replace have_html_espaced_body_text after 517598ba | Takuya Noguchi | |
2018-06-03 | Revert "Add a new have_html_escaped_body_text that match an HTML-escaped text" | Takuya Noguchi | |
This reverts commit 517598ba10793efa02cb90379f78ab97c9c5b25d. | |||
2018-05-31 | Fix merge request unmergeable notification email plural interpolation | Mark Chao | |
2018-05-24 | Merge branch 'mr-conflict-notification' into 'master' | Douwe Maan | |
MR unmergeable notification See merge request gitlab-org/gitlab-ce!18042 | |||
2018-05-18 | Plural reason(s) | Mark Chao | |
2018-05-17 | Resolve "Opening Project with invite but without accepting leads to 404 ↵ | 🙈 jacopo beschi 🙉 | |
error page" | |||
2018-05-17 | Notify with email when merge request became unmergeable | lulalala | |
Display MR unmergeable reasons | |||
2018-05-07 | Backport 5480-epic-notifications from EE | Mario de la Ossa | |
2018-04-08 | Fix undefined method `one?` when pushing to an existing merge request | Stan Hu | |
An untested code path was triggering an Exception because Fixnum doesn't have `one?` implemented in Rails, while arrays and collections do. Closes #45152 | |||
2018-04-04 | Add custom additonal email text to all emails | Eric Eastwood | |
Fix https://gitlab.com/gitlab-org/gitlab-ee/issues/4474 Conflicts: db/schema.rb ee/app/controllers/ee/admin/application_settings_controller.rb ee/app/helpers/ee/application_settings_helper.rb ee/app/models/ee/application_setting.rb ee/app/models/license.rb ee/app/views/layouts/service_desk.html.haml ee/app/views/notify/approved_merge_request_email.html.haml ee/app/views/notify/service_desk_new_note_email.text.erb ee/app/views/notify/service_desk_thank_you_email.text.erb ee/app/views/notify/unapproved_merge_request_email.html.haml ee/lib/ee/api/entities.rb ee/spec/controllers/admin/application_settings_controller_spec.rb ee/spec/models/application_setting_spec.rb ee/spec/requests/api/settings_spec.rb lib/api/settings.rb spec/mailers/previews/notify_preview.rb | |||
2018-04-03 | Fix body of email when commits are pushed to an MR | Sean McGivern | |
This was sending the current user, which is the recipient! It should be the user who pushed the commits. | |||
2018-03-28 | Fix unapproved unassigned MR email erroring out | Eric Eastwood | |
Fix https://gitlab.com/gitlab-org/gitlab-ee/issues/3092 Conflicts: ee/app/views/notify/unapproved_merge_request_email.html.haml spec/mailers/notify_spec.rb | |||
2018-03-05 | Replace deprecated name_with_namespace with full_name in app and spec | Dmitriy Zaporozhets | |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | |||
2018-02-23 | Add DNS verification to Pages custom domains | Nick Thomas | |
2018-02-19 | Resolve "group request membership mail with too long list of "To:"" | 🙈 jacopo beschi 🙉 | |
2018-01-31 | Enable RuboCop Style/RegexpLiteral | Takuya Noguchi | |
2018-01-17 | Initial work to add notification reason to emails | Mario de la Ossa | |
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. | |||
2017-12-22 | Replace '.team << [user, role]' with 'add_role(user)' in specs | blackst0ne | |
2017-12-13 | Make discussion mail References all notes in the discussion | Toon Claes | |
When a note is part of a discussion, the email sent out will be `In-Reply-To` the previous note in that discussion. It also `References` all the previous notes in that discussion, and the original issue. Closes gitlab-org/gitlab-ce#36054. | |||
2017-12-01 | Stop calling #strip in a commit title in Notify specs | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-11-13 | Fix image diff notes email | Felipe Artur | |
2017-10-09 | Don't show an "Unsubscribe" link in snippet comment notifications | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
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-09-01 | Remove some empty lines from notify specs | Grzegorz Bizon | |
2017-09-01 | Optimize notification emails specs | Grzegorz Bizon | |
2017-08-11 | Enable the RSpec/HookArgument cop and auto-correct offenses | Robert Speicher | |
2017-08-03 | Change all `:empty_project` to `:project` | Robert Speicher | |
2017-07-27 | notification email on add new gpg key | Alexis Reigel | |
2017-07-05 | Create and use project path helpers that only need a project, no namespace | Douwe Maan | |
2017-06-21 | Enable Style/DotPosition Rubocop :cop: | Grzegorz Bizon | |
2017-06-14 | Correct RSpec/SingleLineHook cop offenses | Robert Speicher | |
2017-05-25 | Bugfix: Always use the default language when generating emails. | Ruben Davila | |
There was a race condition issue when the application was generating an email and was using a language that was previously being used in other request. | |||
2017-05-04 | Backport of multiple_assignees_feature [ci skip] | Valery Sizov | |
2017-05-01 | Auto-correct `RSpec/DescribedClass` violations | Robert Speicher | |
2017-04-26 | Enable RSpec/FilePath cop | Sean McGivern | |
- Ignore JS fixtures - Ignore qa directory - Rewrite concern specs to put concern name first | |||
2017-04-06 | Merge branch 'update-droplab-to-webpack-version' into new-resolvable-discussion | Luke "Jared" Bennett | |
2017-04-06 | Make it possible to preview pipeline success/failed emails | Toon Claes | |
Visit `/rails/mailers/notify` on your local running GitLab instance to show a preview pipeline success emails. | |||
2017-04-05 | Remove unused code | Douwe Maan | |
2017-04-05 | Address review comments | Douwe Maan | |
2017-04-05 | Fix specs | Douwe Maan | |
2017-04-05 | Add specs | Douwe Maan | |
2017-04-05 | Better notification emails for notes and (diff) discussions | Douwe Maan | |
2017-04-03 | Don't use FFaker in factories, use sequences instead | Rémy Coutable | |
FFaker can generate data that randomly break our test suite. This simplifies our factories and use sequences which are more predictive. Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-03-28 | Use `:empty_project` where possible in mailer specs | Robert Speicher | |
2017-03-22 | Combine redundant `it` blocks in Notify spec | Robert Speicher | |
We perform a bunch of setup for most of these cases, and it didn't make sense to do an entirely new costly setup just to test a different string in the same body of the email we just generated in the last test. |