FEATURE | add push notifications for close to end tasks #10

Merged
Vitalik merged 6 commits from feature/task-notifications into stage 8 months ago
STRIKE commented 8 months ago
Collaborator

Web app. Пуш нотифікаці нагадування про задачі строк яких сплине незабаром

Web app. Пуш нотифікаці нагадування про задачі строк яких сплине незабаром
STRIKE added 2 commits 8 months ago
STRIKE added 1 commit 8 months ago
Vitalik reviewed 8 months ago
}
}
protected async findTasksCloseToEndDate() {
Owner

Максимально не оптимізована функція, можна легко покласти сервер.

  1. Винести в окремий клас.
  2. Переробити на рекурсію, взяли 20 тасок з бази, обробили їх, відправили пуші, взяли 20 наступних, так поки нне закінчаться усі таски. Кількість одночасних тасок які можуть бути в роботі має налаштовуватись з .env.
  3. Між кожним циклом рекурсії додати proccess.nextTick()
  4. Нормалізувати саму функції, позбутись магічних чисел, рознести на підметоди
Максимально не оптимізована функція, можна легко покласти сервер. 1. Винести в окремий клас. 2. Переробити на рекурсію, взяли 20 тасок з бази, обробили їх, відправили пуші, взяли 20 наступних, так поки нне закінчаться усі таски. Кількість одночасних тасок які можуть бути в роботі має налаштовуватись з .env. 3. Між кожним циклом рекурсії додати proccess.nextTick() 4. Нормалізувати саму функції, позбутись магічних чисел, рознести на підметоди
STRIKE marked this conversation as resolved
Vitalik reviewed 8 months ago
export const TASKS_DOCUMENTS_REPOSITORY = Symbol('TASKS_DOCUMENTS_REPOSITORY')
export const TASKS_FAVORITES_REPOSITORY = Symbol('TASKS_FAVORITES_REPOSITORY')
export const PERCENT_OF_TASK_TOTAL_DURATION = 20
Owner

має братись з env

має братись з env
STRIKE marked this conversation as resolved
Vitalik reviewed 8 months ago
const remainingDays = moment(task.endDate).diff(date, 'days')
const taskDeadlineThreshold = Math.ceil(
totalDuration * (Number(getEnv('PERCENT_OF_TASK_TOTAL_DURATION')) / 100),
Owner

не треба getEnv використовувати тут, конфігурація приходить в модуль

не треба getEnv використовувати тут, конфігурація приходить в модуль
STRIKE marked this conversation as resolved
STRIKE added 1 commit 8 months ago
STRIKE added 1 commit 8 months ago
STRIKE added 1 commit 8 months ago
Vitalik merged commit 1b8d07921d into stage 8 months ago
Vitalik deleted branch feature/task-notifications 8 months ago
The pull request has been merged as 1b8d07921d.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.