Skip to content

Add new data to project in push, issue, merge-request and note webhooks data

username-removed-128633 requested to merge bugagazavr/gitlab-ce-extend-hooks into master

Originally opened at !2738 (closed) by @bugagazavr.

What does this MR do?

Adds more data to hooks, makes repository data in all hook more dry ( use one hook_attrs method for repository data in all hooks )

Are there points in the code the reviewer needs to double check?

  1. https://gitlab.com/bugagazavr/gitlab-ce/blob/extend-hooks/app/models/project.rb#L738
  2. https://gitlab.com/bugagazavr/gitlab-ce/blob/extend-hooks/app/models/concerns/issuable.rb#L132
  3. https://gitlab.com/bugagazavr/gitlab-ce/blob/extend-hooks/lib/gitlab/push_data_builder.rb#L52
  4. https://gitlab.com/bugagazavr/gitlab-ce/blob/extend-hooks/lib/gitlab/note_data_builder.rb#L56

Why was this MR needed?

More information about user for push events, and more data about repositiry ( project )

What are the relevant issue numbers?

No

Screenshots (if relevant)

No

Merge request reports