gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.

Age	Commit message (Collapse)	Author
2018-06-07	Apply notification settings level of groups to all child objects	Felipe Artur

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-25	Move NotificationService calls to Sidekiq	Sean McGivern
	The NotificationService has to do quite a lot of work to calculate the recipients for an email. Where possible, we should try to avoid doing this in an HTTP request, because the mail are sent by Sidekiq anyway, so there's no need to schedule those emails immediately. This commit creates a generic Sidekiq worker that uses Global ID to serialise and deserialise its arguments, then forwards them to the NotificationService. The NotificationService gains an `#async` method, so you can replace: notification_service.new_issue(issue, current_user) With: notification_service.async.new_issue(issue, current_user) And have everything else work as normal, except that calculating the recipients will be done by Sidekiq, which will then schedule further Sidekiq jobs to send each email.
2018-04-17	Make issue due email more consistent with other mailers	Sean McGivern

2018-03-30	Only send issue due emails to participants and custom subscribers	Sean McGivern

2018-03-30	Merge branch 'master' into stuartnelson3/gitlab-ce-stn/issue-due-email	Sean McGivern

2018-03-30	Send issue due emails to all participants	Sean McGivern

2018-03-26	Send notification emails when push to a merge request	YarNayar
	Closes #23460
2018-03-19	Always notify new mentions even if explicitly unsubscribed	Mario de la Ossa

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	Make user/author use project.creator in most factories	Rémy Coutable
	Signed-off-by: Rémy Coutable <remy@rymai.me>
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-04	Fix watch level for mentions in description	Sean McGivern
	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.
2017-11-16	Adds Rubocop rule for line break after guard clause	Jacopo
	Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.
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-09-15	created services for keys	haseeb

2017-09-07	Removes default scope from sortable	Tiago Botelho

2017-08-22	rm a comment that is a lie	http://jneen.net/

2017-08-22	add a spec for a participant with a :custom setting	http://jneen.net/
	where that custom setting is not enabled.
2017-08-12	switch to multi-line before block	http://jneen.net/

2017-08-12	restructure the #new_key notification spec	http://jneen.net/

2017-08-12	reset_delivered_emails before testing #new_key	http://jneen.net/
	since the mere act of creating the key sends an email :\|
2017-08-12	move the member spec to be with the other ones	http://jneen.net/
	and add one
2017-08-12	add a spec for new_group_member	http://jneen.net/

2017-08-12	add a spec for never emailing the ghost user	http://jneen.net/

2017-08-12	check notifiability for more emails	http://jneen.net/

2017-08-11	Enable the RSpec/HookArgument cop and auto-correct offenses	Robert Speicher

2017-08-09	Enable the Layout/SpaceBeforeBlockBraces cop	Rémy Coutable
	Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-08-04	Merge branch 'refactor.notification-recipient-builders' into 'master'	Sean McGivern
	Refactor.notification recipient builders See merge request !13197
2017-08-03	add builder helpers with levels and start to separate out #filter!	http://jneen.net/

2017-08-03	Don't include EmailHelpers manually, pick with rspec	Lin Jen-Shin
	`:mailer` is needed to pick it easily, while `type: :mailer` is needed for picking it automatically for tests located in spec/mailers/*_spec.rb It's a bit complicated in spec/services/notification_service_spec.rb but we'll leave it alone for now.
2017-08-03	Change all `:empty_project` to `:project`	Robert Speicher

2017-07-28	De-duplicate two specs in spec/services/notification_service_spec	Robert Speicher
	There were two specs that were testing the exact same thing as the spec above it, but with additional expectations. So we opted to keep the "more expectations" tests and deleted the duplicates. We've also deleted one nested `before` block that was duplicating setup of its parent context.
2017-07-27	Merge branch 'feature/gpg-signed-commits' into 'master'	Dmitriy Zaporozhets
	GPG signed commits Closes #20268 See merge request !9546
2017-07-27	notification email on add new gpg key	Alexis Reigel

2017-07-27	Use described_class when possible	Rémy Coutable
	Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-07-27	Remove superfluous lib: true, type: redis, service: true, models: true, ↵	Rémy Coutable
	services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-07-17	Cache Note#notable for commits and fix tests	Lin Jen-Shin

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/TrailingCommaInLiteral cop	Rémy Coutable
	Use the EnforcedStyleForMultiline: no_comma option. Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-05-04	Backport of multiple_assignees_feature [ci skip]	Valery Sizov

2017-04-09	Fix email not being sent after project export	mhasbini

2017-04-05	Don't use original_discussion_id	Douwe Maan

2017-04-05	Add option to start a new discussion on an MR	Douwe Maan

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-28	Use `:empty_project` where possible in service specs	Robert Speicher