Skip to content

Added user avatar and more project data to hooks

Kirill Zaitsev requested to merge (removed):extend-hooks into master

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

Edited by 🤖 GitLab Bot 🤖

Merge request reports