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

Use useToJSON option of flat #196

Closed
wants to merge 2 commits into from
Closed

Use useToJSON option of flat #196

wants to merge 2 commits into from

Conversation

juanjoDiaz
Copy link
Collaborator

Leverages https://github.com/hughsk/flat/pull/64/files to fix #175

It shouldn't be merged until flat's PR is accepted and merge.

@coveralls
Copy link

coveralls commented Sep 11, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 8314513 on juanjoDiaz:feature/use_useToJSON_option_of_flat into e8b6f6b on zemirco:master.

@coveralls
Copy link

coveralls commented Sep 11, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 02e3c3b on juanjoDiaz:feature/use_useToJSON_option_of_flat into e8b6f6b on zemirco:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 02e3c3b on juanjoDiaz:feature/use_useToJSON_option_of_flat into e8b6f6b on zemirco:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 02e3c3b on juanjoDiaz:feature/use_useToJSON_option_of_flat into e8b6f6b on zemirco:master.

@knownasilya
Copy link
Collaborator

Excellent will wait on the merge.

@knownasilya
Copy link
Collaborator

@juanjoDiaz
Copy link
Collaborator Author

Superseded by #259

@juanjoDiaz juanjoDiaz closed this Feb 25, 2018
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.

Don't flat ObjectId's
3 participants