diff options
author | Stan Hu <stanhu@gmail.com> | 2016-01-04 15:39:52 +0000 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2016-01-04 15:39:52 +0000 |
commit | 561acdebf841f1a6821dc33d02f88192b5fdacc5 (patch) | |
tree | 547a6c97f7121241b7f5507fc450f5d38099a22d /lib/api/projects.rb | |
parent | 01e1b6bd93686fa2d6fa6e45f50d4113dbca6c7a (diff) | |
parent | 79ec7f289748ca5812d51c3a61e3a2f9c2464fda (diff) | |
download | gitlab-ce-561acdebf841f1a6821dc33d02f88192b5fdacc5.tar.gz |
Merge branch 'extra-system-hooks' into 'master'
Added system hooks messages for renaming and transferring a project
This is work in progress but I wanted to get some feedback on it in case there is something I've missed or a better way to do it.
As the title says when a project is renamed or transferred to another group a system hook message is generated.
Currently adds a couple of extra properties to the standard project system hooks messages.
old_path_with_namespace - The old path of the project so we can see what has been renamed / transferred
changed_at - The time that change took place
My main concern is if there is a better way of doing this than passing the extra_data variable into the system hook. Would it be better to extend the project model with these 2 items as transient properties as I doubt they would need to be saved to the database? The other
I've also seen, but can't find, someone talking about the emails sent when a project is renamed or transferred doesn't reference the original project path so if you have similarly named projects then you might not know what has been moved. I think the extra_data (or however it is best to implement this) would allow the extra information to be sent in emails.
Closes [System hooks for project transfer and repository rename](http://feedback.gitlab.com/forums/176466-general/suggestions/6006233-system-hooks-for-project-transfer-and-repository-r)
See merge request !611
Diffstat (limited to 'lib/api/projects.rb')
0 files changed, 0 insertions, 0 deletions