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(Statement Deletion): Fixes store recounts when clients have no LRS attached (LL-322) #1515

Merged
merged 7 commits into from
Feb 24, 2020

Conversation

PrinceWaune
Copy link
Contributor

@PrinceWaune PrinceWaune commented Feb 5, 2020

Transfer the logic of recounts on single statement deletion from
express-restify-mongoose to a post remove hook.

Closes LL-322
@LearningLocker LearningLocker deleted a comment from lgtm-com bot Feb 5, 2020
@LearningLocker LearningLocker deleted a comment from lgtm-com bot Feb 5, 2020
Yevgenii Sharpinskii added 2 commits February 5, 2020 19:04
@crazy-grizzly
Copy link
Contributor

@PrinceWaune , could you please annotate code inside lrs.js like it done in personasImport.js using JSDoc?

@PrinceWaune PrinceWaune closed this Feb 7, 2020
@PrinceWaune PrinceWaune reopened this Feb 7, 2020
@PrinceWaune
Copy link
Contributor Author

@PrinceWaune , could you please annotate code inside lrs.js like it done in personasImport.js using JSDoc?

Sure, no problem!

Copy link
Member

@ryasmi ryasmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @PrinceWaune, looks good, just one thing to check and probably change.

lib/models/lrs.js Show resolved Hide resolved
lib/models/lrs.js Show resolved Hide resolved
@ryasmi ryasmi changed the title Recounts on single statement deletion[LL-322] fix(Statement Deletion): Fixes store recounts when clients have no LRS attached (LL-322) Feb 24, 2020
@ryasmi ryasmi merged commit 8ce4e3b into master Feb 24, 2020
@ryasmi ryasmi deleted the LL-322 branch February 24, 2020 10:36
@HT2Bot
Copy link
Member

HT2Bot commented Feb 24, 2020

🎉 This PR is included in version 6.0.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants