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.

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

Clone Proto to modify and use the copy. #101

Closed
wants to merge 1 commit into from

Conversation

marshallswain
Copy link
Member

As mentioned in feathersjs-ecosystem/feathers-mongodb#9, this creates a clone of Proto to be modified.

@daffl
Copy link
Member

daffl commented Jan 8, 2015

That's a great idea. Tests are failing, probably because they need Proto.create now though.

@marshallswain
Copy link
Member Author

I don't think this is a good idea. Even after removing the Proto.isPrototypeOf() test, EVERY test after the first 10 fails.

@daffl
Copy link
Member

daffl commented Jan 9, 2015

It is the right approach though I think. I'll give it a try as well and see what is happening.

@marshallswain marshallswain deleted the clone-proto branch April 7, 2015 15:01
daffl pushed a commit that referenced this pull request Aug 29, 2018
Finalizing client side authentication module
daffl pushed a commit that referenced this pull request Aug 29, 2018
Finalizing client side authentication module
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.

2 participants