Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

added apps:transfer #13

Merged
merged 3 commits into from
Feb 12, 2016
Merged

added apps:transfer #13

merged 3 commits into from
Feb 12, 2016

Conversation

jdx
Copy link
Contributor

@jdx jdx commented Jan 29, 2016

No description provided.

@ransombriggs
Copy link
Contributor

@dickeyxxx before this is merged, we need to implement the --locked flag for org transfers, see heroku/legacy-cli#1896

@raulb
Copy link
Contributor

raulb commented Feb 12, 2016

For the record. With this change we're deprecating the command heroku sharing:transfer:

$ heroku sharing:transfer ORG-NAME -a APP-NAME
 ▸    This command is now heroku apps:transfer

@raulb raulb mentioned this pull request Feb 12, 2016
4 tasks
@raulb
Copy link
Contributor

raulb commented Feb 12, 2016

@ransombriggs 7ba5400

$ heroku apps:transfer organization-name-test -a sleepy-mesa-53642 --locked
Transferring sleepy-mesa-53642 to organization-name-test... done
Locking sleepy-mesa-53642... done

Though, not sure what's the best way to unit test that. Recommendations?

@ransombriggs
Copy link
Contributor

@raul I added a unit test for the --locked flag on apps:transfer in cb67c91

@raulb
Copy link
Contributor

raulb commented Feb 12, 2016

@ransombriggs awesome!, thanks!

raulb added a commit that referenced this pull request Feb 12, 2016
@raulb raulb merged commit 421fc42 into master Feb 12, 2016
@raulb raulb deleted the sharing branch February 12, 2016 17:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants