summaryrefslogtreecommitdiff
path: root/app/mailers
Commit message (Collapse)AuthorAgeFilesLines
* Fix body of email when commits are pushed to an MRSean McGivern2018-04-031-0/+1
| | | | | This was sending the current user, which is the recipient! It should be the user who pushed the commits.
* Send notification emails when push to a merge requestYarNayar/gitlab-ce-23460-send-email-when-pushing-more-commits-to-the-merge-requestYarNayar2018-03-261-0/+8
| | | | Closes #23460
* Use Project#full_name instead of name_with_namespaceDmitriy Zaporozhets2018-03-071-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Add DNS verification to Pages custom domainsNick Thomas2018-02-232-0/+44
|
* Resolve "group request membership mail with too long list of "To:""🙈 jacopo beschi 🙉2018-02-191-9/+2
|
* use Gitlab::UserSettings directly as a singleton instead of ↵Mario de la Ossa2018-02-022-7/+3
| | | | including/extending it
* Initial work to add notification reason to emailsMario de la Ossa2018-01-163-34/+38
| | | | | | | | | | | 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.
* Make discussion mail References all notes in the discussiontc-correct-email-in-reply-toToon Claes2017-12-131-4/+4
| | | | | | | | | 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.
* Make mail notifications of discussion notes In-Reply-To of each otherToon Claes2017-12-132-5/+17
| | | | | | | When a note is part of a discussion, the email sent out should be `In-Reply-To` the previous note in that discussion. Closes gitlab-org/gitlab-ce#36054
* Send a confirmation email when the user adds a secondary email address. ↵Brett Walker2017-09-231-6/+0
| | | | Utilizes the Devise `confirmable` capabilities. Issue #37385
* `current_application_settings` belongs on `Gitlab::CurrentSettings`Sean McGivern2017-08-311-1/+3
| | | | | | | | | | | | | | | | The initializers including this were doing so at the top level, so every object loaded after them had a `current_application_settings` method. However, if someone had rack-attack enabled (which was loaded before these initializers), it would try to load the API, and fail, because `Gitlab::CurrentSettings` didn't have that method. To fix this: 1. Don't include `Gitlab::CurrentSettings` at the top level. We do not need `Object.new.current_application_settings` to work. 2. Make `Gitlab::CurrentSettings` explicitly `extend self`, as we already use it like that in several places. 3. Change the initializers to use that new form.
* Speed up Group#user_ids_for_project_authorizationsNick Thomas2017-08-141-2/+2
|
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-011-1/+1
|
* find_by_id -> find_by(:id, ...)Alexis Reigel2017-07-271-2/+2
|
* notification email on add new gpg keyAlexis Reigel2017-07-271-0/+10
|
* Create and use project path helpers that only need a project, no namespaceDouwe Maan2017-07-055-10/+11
|
* use common devise layout and use heading styleAlexis Reigel2017-06-131-1/+1
|
* use common layout for devise mailerAlexis Reigel2017-06-131-1/+3
|
* Bugfix: Always use the default language when generating emails.32748-emails-are-being-sent-with-the-wrong-languageRuben Davila2017-05-251-0/+6
| | | | | | 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.
* Backport of multiple_assignees_feature [ci skip]Valery Sizov2017-05-041-2/+4
|
* Rename GitlabMarkdownHelper -> MarkupHelperToon Claes2017-04-271-1/+1
| | | | It does more than solely GitLab Markdown, so rename to MarkupHelper.
* Address review commentsDouwe Maan2017-04-051-9/+7
|
* Address review commentsDouwe Maan2017-04-051-5/+0
|
* Fix specsDouwe Maan2017-04-052-2/+3
|
* Better notification emails for notes and (diff) discussionsDouwe Maan2017-04-051-2/+5
|
* Enable Style/Proc cop for rubocopmhasbini2017-04-021-2/+2
|
* All CI offline migrationsKamil Trzciński2017-03-172-31/+0
|
* Fix code for copsDouwe Maan2017-02-231-5/+6
|
* Enable Style/ConditionalAssignmentDouwe Maan2017-02-231-3/+3
|
* extract pipeline mails layoutAlexis Reigel2017-02-221-2/+2
|
* Update Rubocop to ruby 2.3zj-drop-ruby-21-testsZ.J. van de Weg2017-02-071-1/+1
|
* Merge branch 'no_project_notes' into 'master' Sean McGivern2017-01-251-0/+8
|\ | | | | | | | | Support notes without a project (personal snippets notes) See merge request !8468
| * Support notes without projectJarka Kadlecova2017-01-181-0/+8
| |
* | Handles unsubscribe from notifications via emailPawel Chojnacki2017-01-131-6/+14
|/ | | | | - allows unsubscription processing of email in format "reply+%{key}+unsubscribe@acme.com" (example) - if config.address includes %{key} and replies are enabled every unsubscriable message will include mailto: link in its List-Unsubscribe header
* Add new template to handle both commit & mr noteshhoopes2016-11-251-0/+2
| | | | | | | | Currently comments on commits and merge requests do not require merge request- or commit-specific information, but can use the same template. Rather than change the method which calls the template, I opted to keep the templates separate and create a new template to highlight their identicality, while preserving the option to distinguish them from each other in the future. Also removed some of the inconsistencies between text and html email versions. Still needed is a text-only version of git diffs and testing.
* Merge remote-tracking branch 'upstream/master' into pipeline-notificationsLin Jen-Shin2016-11-082-8/+8
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (70 commits) Fix routing spec for group controller Add small improvements to constrainers and specs Faster search Fix broken commits search Changed helper method to check for none on params Moved if statements around in view API: Return 400 when creating a systemhook fails Update non-exist group spinach test to match routing Bump omniauth-gitlab to 1.0.2 to fix incompatibility with omniauth-oauth2 Replace trigger with the new ID of the docs project Refactor method name 17492 Update link color for more accessible contrast Fixed todos empty state when filtering Refactor namespace regex implements reset incoming email token on issues modal and account page, reactivates all tests and writes more tests for it Use separate email-friendly token for incoming email and let incoming email token be reset Use the Gitlab Workhorse HTTP header in the admin dashboard Refactor project routing Fix 404 when visit /projects page Rewritten spinach git_blame tests to rspec feature tests Add tests for project#index routing ...
| * Use method helper instead of add_template_helperbogdanvlviv2016-11-062-8/+8
| |
* | Use bcc for pipeline emails because:Lin Jen-Shin2016-10-181-6/+11
|/ | | | | | 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.
* Just show the first merge request we found, feedback:Lin Jen-Shin2016-10-141-1/+1
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019#note_16963209
* Introduce Pipeline#merge_requests_with_active_first,Lin Jen-Shin2016-10-141-4/+1
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019#note_16956802
* skip the premailer compilation step since css is already inlined for the ↵Mike Greiling2016-10-081-1/+1
| | | | pipeline emails
* prevent pipeline emails from using the normal layout wrapperMike Greiling2016-10-071-1/+4
|
* include pipeline ref in subject lineMike Greiling2016-10-061-3/+3
|
* update subject and email header messageMike Greiling2016-10-061-2/+4
|
* Merge remote-tracking branch 'upstream/master' into pipeline-emailsLin Jen-Shin2016-10-043-2/+11
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (372 commits) Enable Lint/StringConversionInInterpolation cop and autocorrect offenses resolve duplicated changelog entry credit myself :smile: change determine conditions override subject method in devise mailer follow the styleguide: Don't use parentheses around a literal wrap subject with method subject move spec back into shared example `an email sent from GitLab` stub config settings in spec remove empty line at block body end remove extra entry create new test in `spec/mailers/notify_spec.rb` move changelog to 8.13 add configurable email subject suffix Fixes sidebar navigation. Convert "SSH Keys" Spinach features to RSpec Enable import/export back for non-admins Update gitlab-shell to 3.6.3 Updated artwork of empty group state. Better empty state for Groups view. ...
| * change determine conditionsFu Xu2016-10-032-2/+2
| |
| * override subject method in devise mailerFu Xu2016-10-031-0/+8
| |
| * wrap subject with method subjectFu Xu2016-10-031-1/+1
| |
| * add configurable email subject suffixFu Xu2016-10-031-0/+1
| |
| * Wrap List-Unsubscribe link in angle bracketsfix-unsubscribe-headerSean McGivern2016-09-261-1/+1
| |
View Lorry Controller Status