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

Mongo errors 624 #627

Merged
merged 10 commits into from
Oct 20, 2023
Merged

Mongo errors 624 #627

merged 10 commits into from
Oct 20, 2023

Conversation

weissu42
Copy link
Member

resolves #624

weissu42 and others added 5 commits August 18, 2023 13:15
* remove unused method in MongoSchedule

Signed-off-by: Ute Weiss <[email protected]>
Signed-off-by: Yani Kolev <[email protected]>
…hout spamming mongo errors

* clean up SchedulesRepository methods to not take scheduleName or scheduleId as parameters (these are known in repo anyway)
* introduce ScheduleState for clean implementation of the fact whether this instance, another instance or no instance is active for a schedule

Signed-off-by: Yani Kolev <[email protected]>
Signed-off-by: Ute Weiss <[email protected]>
@weissu42 weissu42 marked this pull request as ready for review September 15, 2023 09:51
src/repository/SchedulesRepository.ts Show resolved Hide resolved
src/schedule/SchedulePing.ts Outdated Show resolved Hide resolved
@YaniKolev YaniKolev merged commit 2dc574f into main Oct 20, 2023
15 checks passed
@YaniKolev YaniKolev deleted the mongo-errors-624 branch December 15, 2023 09:38
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.

Momo generates a lot of Mongo Errors
2 participants