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

Feature/editors availability check #795

Merged
merged 21 commits into from
Jul 11, 2023
Merged

Conversation

rivexe
Copy link
Member

@rivexe rivexe commented Mar 22, 2023

Background job that checks the availability of editors.

  1. EditorsCheck contains logic of task.
  2. Also divided the logic of building notifications by subject in Notifier.
  3. If the setting editors_check_interval is set to 0, no background job will be done.

@rivexe rivexe requested a review from Antipkin-A March 22, 2023 11:26
@rivexe rivexe force-pushed the feature/editors-availability-check branch from 9c4ff4f to 6637244 Compare March 28, 2023 06:57
@rivexe rivexe force-pushed the feature/editors-availability-check branch from 8ac09e4 to aff416e Compare April 5, 2023 11:51
@rivexe rivexe force-pushed the feature/editors-availability-check branch from a6f09e3 to 35c6d73 Compare July 11, 2023 08:44
@LinneyS LinneyS merged commit 40f22cf into develop Jul 11, 2023
3 of 4 checks passed
@LinneyS LinneyS deleted the feature/editors-availability-check branch July 11, 2023 12:55
@pierreozoux
Copy link
Contributor

FYI, we will disable this check for us:
#880

I think it is a nice idea, but currently, it is disabling our app, without really knowing why / or how to fix the flackiness.

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.

3 participants