Skip to content

Add possibility to copy own twiddle#215

Merged
rwjblue merged 1 commit intoember-cli:masterfrom
pangratz:copy-own-twiddle
Oct 12, 2015
Merged

Add possibility to copy own twiddle#215
rwjblue merged 1 commit intoember-cli:masterfrom
pangratz:copy-own-twiddle

Conversation

@pangratz
Copy link
Copy Markdown
Contributor

If a gist owned by the authenticated user is shown, a new menu item
"Copy Twiddle" allows to copy the current files into a new Twiddle. Note
that the newly created Twiddle is not yet saved.

By this, Twiddles (e.g. Ember setup for a specific version) can be used
as a starting point without having to re-setup the whole Twiddle again
and again.


This addresses #78 and #157 (comment).

If a gist owned by the authenticated user is shown, a new menu item
"Copy Twiddle" allows to copy the current files into a new Twiddle. Note
that the newly created Twiddle is not yet saved.

By this, Twiddles (e.g. Ember setup for a specific version) can be used
as a starting point without having to re-setup the whole Twiddle again
and again.
@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Oct 12, 2015

👍 - LGTM

rwjblue added a commit that referenced this pull request Oct 12, 2015
Add possibility to copy own twiddle
@rwjblue rwjblue merged commit 122d9f8 into ember-cli:master Oct 12, 2015
@pangratz
Copy link
Copy Markdown
Contributor Author

Aaaand it's live on http://canary.ember-twiddle.com. Oh boy, this workflow is awesome! 👯

@pangratz pangratz deleted the copy-own-twiddle branch October 12, 2015 12:41
@rwjblue rwjblue mentioned this pull request Oct 12, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really cool test!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thx! But it currently misses assertions that all files have been copied correctly. Do you have any suggestions? Would it be sufficient to run the twiddle and check if the output is the same as the original one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants