Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rake task to transfer procedure ownership #2756

Merged
merged 3 commits into from
Oct 10, 2018
Merged

Rake task to transfer procedure ownership #2756

merged 3 commits into from
Oct 10, 2018

Conversation

fredZen
Copy link
Contributor

@fredZen fredZen commented Oct 4, 2018

No description provided.

@fredZen fredZen force-pushed the frederic/transfer_procedure_ownership branch from 5d529b1 to 004059a Compare October 4, 2018 16:31
gregoirenovel
gregoirenovel approved these changes Oct 8, 2018
View reviewed changes
Copy link
Contributor

@gregoirenovel gregoirenovel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je trouve la tache un peu grosse, je me demande si j'aurais pas eu tendance à splitter en deux taches. En particulier, ce que je trouve dur à comprendre rapidement est la bonne manière d'invoquer la tache dans les deux cas.

@fredZen
Copy link
Contributor Author

fredZen commented Oct 8, 2018

Je vois ce que tu veux dire, oui.

@fredZen fredZen force-pushed the frederic/transfer_procedure_ownership branch from 004059a to 88487bd Compare October 10, 2018 03:36
@fredZen fredZen merged commit fbdb3ad into dev Oct 10, 2018
@fredZen fredZen deleted the frederic/transfer_procedure_ownership branch October 10, 2018 03:47
@fredZen fredZen mentioned this pull request Oct 10, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gregoirenovel gregoirenovel gregoirenovel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fredZen @gregoirenovel