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

fix rangeEnd usingDotNotation #722

Merged
merged 1 commit into from
Nov 24, 2018

Conversation

Moocar
Copy link

@Moocar Moocar commented Nov 21, 2018

I left out a usingDotNotation on calculateRangeEnd in #718. Fixed and a test added.

@obeliskos
Copy link
Collaborator

Great, thanks again!

@obeliskos obeliskos merged commit bb96365 into techfort:master Nov 24, 2018
@Moocar
Copy link
Author

Moocar commented Dec 14, 2018

@obeliskos I hate to be one of those people, but what's the release process for Loki these days? Any rough ETA on when we can expect a new version with these latest changes? Let me know if I can help with testing etc.

@obeliskos
Copy link
Collaborator

Hey @Moocar, I will update the changelog and request @techfort publish to npm.

I expect this will be 1.5.6 version, hopefully within the next few days. Thanks for your patience :)

LokiJS originally targeted solstice/equinox deliveries so your nested indexed queries might be nice for a return to tradition.

@Moocar
Copy link
Author

Moocar commented Dec 19, 2018

Awesome. Thanks @obeliskos and @techfort !

@obeliskos
Copy link
Collaborator

Sorry for the delay, I notified @techfort after updating the changelog, but he has been occupied with personal issues so it seems he has not done this yet.

Hopefully he finds time to publish in the next few days, otherwise I will drop an email to remind him.

@Moocar
Copy link
Author

Moocar commented Jan 21, 2019

Just saw the new release. Thanks so much!

@obeliskos
Copy link
Collaborator

Thanks for the contributions!

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