summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Expand)AuthorAgeFilesLines
* Rename user_can_admin? because it's more accurateadd-star-for-action-scopeLin Jen-Shin2017-08-071-2/+2
* Merge remote-tracking branch 'upstream/master' into add-star-for-action-scopeLin Jen-Shin2017-08-077-301/+274
|\
| * Merge branch 'fix/sm/34547-cannot-connect-to-ci-server-error-messages' into '...Sean McGivern2017-08-041-1/+6
| |\
| | * essentialShinya Maeda2017-08-041-1/+6
| * | Merge branch 'refactor.notification-recipient-builders' into 'master'Sean McGivern2017-08-042-291/+249
| |\ \ | | |/ | |/|
| | * another rubocop style fixrefactor.notification-recipient-buildershttp://jneen.net/2017-08-031-1/+1
| | * rubocop style fixhttp://jneen.net/2017-08-031-1/+1
| | * add a comment warning that note.project may be nilhttp://jneen.net/2017-08-031-0/+4
| | * remove build_relabeled_recipientshttp://jneen.net/2017-08-032-20/+7
| | * move the read_ability logic into NotificationRecipienthttp://jneen.net/2017-08-032-17/+0
| | * rm unused methodshttp://jneen.net/2017-08-031-26/+0
| | * disable the delegate cophttp://jneen.net/2017-08-031-0/+1
| | * deparameterize `project`http://jneen.net/2017-08-032-41/+37
| | * unmemoize read_abilityhttp://jneen.net/2017-08-031-8/+9
| | * rm unused NewNote#subjecthttp://jneen.net/2017-08-031-4/+0
| | * clearer argument namehttp://jneen.net/2017-08-031-2/+2
| | * short-circuit if there is no policy, and add :read_project checkhttp://jneen.net/2017-08-031-5/+1
| | * force queries to include notification settingshttp://jneen.net/2017-08-031-2/+11
| | * use a simple pluck, since equivalent filtering happens laterhttp://jneen.net/2017-08-031-3/+1
| | * style fixeshttp://jneen.net/2017-08-031-1/+2
| | * .notifiable_users: compact the passed-in usershttp://jneen.net/2017-08-031-1/+1
| | * default the project to target.projecthttp://jneen.net/2017-08-032-3/+4
| | * require that the user be able to :read_buildhttp://jneen.net/2017-08-031-1/+2
| | * make sure users have to be able to read_pipelinehttp://jneen.net/2017-08-031-1/+2
| | * use intersection and diff operators instead of eachhttp://jneen.net/2017-08-031-22/+2
| | * make sure #custom_action is always definedhttp://jneen.net/2017-08-031-0/+4
| | * move Recipient to its own NotificationRecipient filehttp://jneen.net/2017-08-031-109/+7
| | * move the #build_* methods to static, parameterize the projecthttp://jneen.net/2017-08-032-41/+40
| | * factor out .notifiable_usershttp://jneen.net/2017-08-032-41/+8
| | * don't elevate to :watch if no @custom_action is providedhttp://jneen.net/2017-08-031-1/+1
| | * rm the @builder argument and factor out .notifiable_usershttp://jneen.net/2017-08-032-15/+38
| | * move filtering logic into Recipient classhttp://jneen.net/2017-08-031-99/+102
| | * add builder helpers with levels and start to separate out #filter!http://jneen.net/2017-08-031-54/+113
| | * make recipients mutative during the buildhttp://jneen.net/2017-08-031-64/+73
| | * move build_custom_key to Defaulthttp://jneen.net/2017-08-031-8/+6
| | * factor out the `target` argument to helpershttp://jneen.net/2017-08-031-19/+19
| | * move the build arguments to the initializershttp://jneen.net/2017-08-031-14/+60
| | * move the builders to classes with a #buildhttp://jneen.net/2017-08-031-219/+257
| | * use notification_setting_for_user_project in reject_usershttp://jneen.net/2017-08-031-18/+2
| | * protect against nil project/group/settinghttp://jneen.net/2017-08-031-4/+4
| | * move notification_setting_for_user_project to a public class methodhttp://jneen.net/2017-08-031-13/+13
| | * move the ability check to reject_users_without_accesshttp://jneen.net/2017-08-031-1/+2
| * | Avoid plucking Todo ids and use sub-queries insteadToon Claes2017-08-032-7/+11
| |/
| * Make Delete Merged Branches handle wildcard protected branches correctlyToon Claes2017-08-021-1/+1
| * Merge branch '35815-webhook-log-encoding-error' into 'master'Dmitriy Zaporozhets2017-08-021-1/+7
| |\
| | * Fix encoding error for WebHook loggingAlexander Randa2017-08-011-1/+7
* | | Merge remote-tracking branch 'upstream/master' into add-star-for-action-scopeLin Jen-Shin2017-08-0290-534/+1278
|\ \ \ | |/ /
| * | Added some extra TODOs for the Legacy Storage refactorGabriel Mazetto2017-08-011-0/+2
| * | Make disk_path keyword argument and optionalGabriel Mazetto2017-08-011-1/+1
| * | Rename more path_with_namespace -> full_path or disk_pathGabriel Mazetto2017-08-011-1/+1