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

Add an ngCopy directive #4172

Closed
wants to merge 1 commit into from
Closed

Add an ngCopy directive #4172

wants to merge 1 commit into from

Conversation

benmccann
Copy link
Contributor

No description provided.

@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@benmccann
Copy link
Contributor Author

I've signed the CLA ( Benjamin McCann )

@petebacondarwin
Copy link
Member

You would need to add cut and paste too I guess

@rodyhaddad
Copy link
Contributor

As well as docs for the new directives

@benmccann
Copy link
Contributor Author

done and done

@ghost ghost assigned petebacondarwin Sep 26, 2013
@petebacondarwin
Copy link
Member

Nice one @benmccann. Looks good. I'll tweak the commit message - no capital first letter or full stop in the subject - and merge.

@benmccann
Copy link
Contributor Author

Thanks! Will this make the 1.2.0 release then?

@petebacondarwin
Copy link
Member

Need confirmation from the core team that they are happy to include this
functionality.

On 26 September 2013 22:24, Ben McCann [email protected] wrote:

Thanks! Will this make the 1.2.0 release then?


Reply to this email directly or view it on GitHubhttps://github.com//pull/4172#issuecomment-25205216
.

@amatiasq
Copy link

+1

@benmccann
Copy link
Contributor Author

@amatiasq this is already released with angular 1.2 i believe

jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
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.

5 participants