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

Support for new field update operators ($min, $max, $currentDate and $mul) #7857

Closed
sebakerckhof opened this issue Oct 2, 2016 · 5 comments
Closed

Comments

@sebakerckhof
Copy link
Contributor

@sebakerckhof sebakerckhof commented Oct 2, 2016

Since mongo 2.6+ there are some new field operators: $min, $max, $currentDate and $mul.

Minimongo currently does not yet support these modifiers. @stubailo encouraged to make a PR for this, which I plan to do. So I mainly create this issue to link the PR to an issue.

@stubailo
Copy link
Contributor

@stubailo stubailo commented Oct 2, 2016

I'd suggest doing it in several smaller PRs, if possible - that way we can merge each individual one faster.

@stubailo stubailo changed the title Feature request: Support for new field update operators ($min, $max, $currentDate and $mul) Support for new field update operators ($min, $max, $currentDate and $mul) Oct 2, 2016
@stubailo
Copy link
Contributor

@stubailo stubailo commented Oct 2, 2016

Oh never mind, I see that you have opened a PR already.

@sebakerckhof
Copy link
Contributor Author

@sebakerckhof sebakerckhof commented Oct 2, 2016

Yeah, I could still split them up if you think it's better, but even with them combined it's a pretty small PR.

@stubailo
Copy link
Contributor

@stubailo stubailo commented Oct 2, 2016

Yep, no problem.

@tmeasday
Copy link
Contributor

@tmeasday tmeasday commented Nov 8, 2016

#7858 is merged!

@tmeasday tmeasday closed this Nov 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.