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

[issue1797] Adding ng-open directive #1829

Closed
wants to merge 1 commit into from
Closed

[issue1797] Adding ng-open directive #1829

wants to merge 1 commit into from

Conversation

davidchang
Copy link
Contributor

No description provided.

@mhevery
Copy link
Contributor

mhevery commented Jan 18, 2013

Thanks for your contribution! In order for us to be able to accept it, we ask you to sign our CLA (contributor's license agreement).

CLA is important for us to be able to avoid legal troubles down the road.

For individuals (a simple click-through form):
http://code.google.com/legal/individual-cla-v1.0.html

For corporations (print, sign and scan+email, fax or mail):
http://code.google.com/legal/corporate-cla-v1.0.html

@davidchang
Copy link
Contributor Author

CLA signed.

@mhevery
Copy link
Contributor

mhevery commented Jan 19, 2013

MERGED

@mhevery mhevery closed this Jan 19, 2013
@davidchang
Copy link
Contributor Author

mhevery@, are you sure that should have merged? The Travis build indicated I had caused a whole bunch of tests to fail, right?

@mhevery
Copy link
Contributor

mhevery commented Jan 23, 2013

yes our builds are green:
http://ci.angularjs.org/job/angular.js-angular-master/

On Tue, Jan 22, 2013 at 9:30 PM, David Chang [email protected]:

mhevery@, are you sure that should have merged? The Travis build
indicated I had caused a whole bunch of tests to fail, right?


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

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.

2 participants