Commit graph

11 commits

Author SHA1 Message Date
http://jneen.net/
38737345ab skip the :read_project check for new_project_member
since we're just adding them as a member, the permission may still
return false.
2017-08-11 16:02:08 -07:00
http://jneen.net/
7e7f602d29 make NotificationRecipient a little more customizable 2017-08-11 16:02:01 -07:00
http://jneen.net/
eaa503d679 move the read_ability logic into NotificationRecipient 2017-08-03 09:07:18 -07:00
http://jneen.net/
e5496e1e8e use the accessor for type 2017-08-03 09:07:18 -07:00
http://jneen.net/
4af2c64713 fix comment 2017-08-03 09:07:18 -07:00
http://jneen.net/
444c858449 use safe navigation on notification_setting 2017-08-03 09:07:18 -07:00
http://jneen.net/
c2dd4239c9 short-circuit if there is no policy, and add :read_project check 2017-08-03 09:07:18 -07:00
http://jneen.net/
18288fe21e style fixes 2017-08-03 09:07:18 -07:00
http://jneen.net/
e7d136ebda don't require project
it's not there in the case of personal snippets, f. ex., and we've
already guarded against its being missing in #find_notification_setting
2017-08-03 09:07:18 -07:00
http://jneen.net/
19309b9705 default the project to target.project 2017-08-03 09:07:18 -07:00
http://jneen.net/
618a3d125c move Recipient to its own NotificationRecipient file 2017-08-03 09:07:18 -07:00