romanov73/ng-tracker
1
0
Fork 0
Code Issues 18 Pull Requests 4 Projects Releases Wiki Activity

Resolve "Генерация периодических задач по тегам" #207

Merged
ashette merged 32 commits from 79-students-tags-generation into dev 2019-05-16 07:30:47 +04:00
Conversation 26 Commits 32 Files Changed 32 +656 -62
ashette commented 2019-04-28 16:30:46 +04:00 (Migrated from gitlab.com)
Copy Link

Closes #79

Closes #79
ashette commented 2019-04-28 16:30:46 +04:00 (Migrated from gitlab.com)
Copy Link

changed milestone to %7

changed milestone to %7
ashette commented 2019-04-28 23:58:59 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>b4c11a7f - #79 year generation</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=40599236&start_sha=a352f561fac089d072be21c25fa7f09d77fd1dc5)
ashette commented 2019-04-30 22:06:09 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>7ce8b208 - #79 fixing bug</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=40808287&start_sha=b4c11a7f90cd8dd6733959a8e15dd7ea478f389c)
ashette commented 2019-05-06 22:39:35 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>65dea9bc - #79 adding task planer generation</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=41254859&start_sha=7ce8b20841e8a16740aa1ec319737ee1283637a6)
ashette commented 2019-05-11 22:59:41 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>0de34c89 - #79 fixing planner and year generation</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=41770622&start_sha=65dea9bce2c211126114386768ed6940fbbaee9f)
ashette commented 2019-05-11 23:06:58 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>c59bb45c - #79 fixing codestyle</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=41770903&start_sha=0de34c89fea802355890f8d0fcecaaf0719c5d51)
ashette commented 2019-05-11 23:26:57 +04:00 (Migrated from gitlab.com)
Copy Link

added 50 commits

  • c59bb45c...ff7f5359 - 49 commits from branch dev
  • 7798508d - Merge branch 'dev' into 79-students-tags-generation

Compare with previous version

added 50 commits <ul><li>c59bb45c...ff7f5359 - 49 commits from branch <code>dev</code></li><li>7798508d - Merge branch &#39;dev&#39; into 79-students-tags-generation</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=41771452&start_sha=c59bb45c70bdab32f1ce88a63006e0cc81c77ece)
ashette commented 2019-05-11 23:33:01 +04:00 (Migrated from gitlab.com)
Copy Link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
ashette commented 2019-05-11 23:33:01 +04:00 (Migrated from gitlab.com)
Copy Link

assigned to @romanov73

assigned to @romanov73
romanov73 commented 2019-05-13 11:10:43 +04:00 (Migrated from gitlab.com)
Copy Link

нужно переместить в утилитарный класс, он уже есть в проекте

нужно переместить в утилитарный класс, он уже есть в проекте
romanov73 commented 2019-05-13 11:12:05 +04:00 (Migrated from gitlab.com)
Copy Link

все из БД вытаскивать слишком затратно, нужно написать запрос

все из БД вытаскивать слишком затратно, нужно написать запрос
romanov73 commented 2019-05-13 11:12:20 +04:00 (Migrated from gitlab.com)
Copy Link

param?

param?
romanov73 commented 2019-05-13 11:13:58 +04:00 (Migrated from gitlab.com)
Copy Link

условие нужно вынести в отдельный метод

условие нужно вынести в отдельный метод
romanov73 commented 2019-05-13 11:15:24 +04:00 (Migrated from gitlab.com)
Copy Link

Зачем нужна эта сущность? Все, что анализируется и так существует в виде записей с атрибутами. Если это задача по расписанию, так она в рантайме выполняется, персистентность не нужна

Зачем нужна эта сущность? Все, что анализируется и так существует в виде записей с атрибутами. Если это задача по расписанию, так она в рантайме выполняется, персистентность не нужна
ashette commented 2019-05-13 20:52:13 +04:00 (Migrated from gitlab.com)
Copy Link

Эта сущность выполняет роль планировщика задач. Он записывает в план задачи, которые необходимо будет создать в будущем по последним дедлайнам. План хранится в бд. То есть каждый день мы запускаем планировщика, он ищет периодические задачи, которые нужно "запланировать", записывает их в план. Если его план не пуст - он создает задачи. Таким образом мы избегаем повторения генерации одних и тех же задач.

Эта сущность выполняет роль планировщика задач. Он записывает в план задачи, которые необходимо будет создать в будущем по последним дедлайнам. План хранится в бд. То есть каждый день мы запускаем планировщика, он ищет периодические задачи, которые нужно "запланировать", записывает их в план. Если его план не пуст - он создает задачи. Таким образом мы избегаем повторения генерации одних и тех же задач.
ashette commented 2019-05-13 20:55:02 +04:00 (Migrated from gitlab.com)
Copy Link

Сделано для разделения задач на годовые и периодические. Делается выборка задач за год, считается количество задач по тегам. Если задач по тегу == 1, то просто создаем такую же в след году. Если задач по тегу больше 2, то это уже периодическая задача, которая генерируется по алгоритму.

Сделано для разделения задач на годовые и периодические. Делается выборка задач за год, считается количество задач по тегам. Если задач по тегу == 1, то просто создаем такую же в след году. Если задач по тегу больше 2, то это уже периодическая задача, которая генерируется по алгоритму.
romanov73 commented 2019-05-13 23:49:45 +04:00 (Migrated from gitlab.com)
Copy Link

хотя бы тогда createPeriodicTask

хотя бы тогда createPeriodicTask
ashette commented 2019-05-15 23:48:06 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=42193617&start_sha=7798508d4f042da97156e1b613a0dfd61e7208e4#cf476a4094e8f74ec155c8dcec84350f740916cf_211_210)
ashette commented 2019-05-15 23:48:06 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=42193617&start_sha=7798508d4f042da97156e1b613a0dfd61e7208e4#cf476a4094e8f74ec155c8dcec84350f740916cf_222_216)
ashette commented 2019-05-15 23:48:06 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=42193617&start_sha=7798508d4f042da97156e1b613a0dfd61e7208e4#cf476a4094e8f74ec155c8dcec84350f740916cf_190_191)
ashette commented 2019-05-15 23:48:06 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>bc9cca62 - #79 refactoring</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=42193617&start_sha=7798508d4f042da97156e1b613a0dfd61e7208e4)
ashette commented 2019-05-16 00:14:54 +04:00 (Migrated from gitlab.com)
Copy Link

added 32 commits

  • bc9cca62...51851baa - 31 commits from branch dev
  • cd1caa55 - Merge branch 'dev' into 79-students-tags-generation

Compare with previous version

added 32 commits <ul><li>bc9cca62...51851baa - 31 commits from branch <code>dev</code></li><li>cd1caa55 - Merge branch &#39;dev&#39; into 79-students-tags-generation</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/80/diffs?diff_id=42195180&start_sha=bc9cca62d68751716b77a75a78c1b6ed6e9de6ad)
ashette commented 2019-05-16 00:22:53 +04:00 (Migrated from gitlab.com)
Copy Link

Всё исправила. Запрос есть, делается выборка не всех тасков, а только за год.

Всё исправила. Запрос есть, делается выборка не всех тасков, а только за год.
romanov73 commented 2019-05-16 07:30:40 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2019-05-16 07:30:47 +04:00 (Migrated from gitlab.com)
Copy Link

merged

merged
romanov73 commented 2019-05-16 07:30:47 +04:00 (Migrated from gitlab.com)
Copy Link

mentioned in commit 19a6b45cf9

mentioned in commit 19a6b45cf9e0841bb7a97b1905a58effbde1bf7f
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend

   
bug

   
frontend

   
выполнить первой

   
выполняется

   
конференции

   
события

   
статьи
No Label
backend
bug
frontend
выполнить первой
выполняется
конференции
события
статьи
Milestone
Clear milestone
No items
No Milestone
Работа со студентами
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: romanov73/ng-tracker#207
No description provided.
Delete Branch "79-students-tags-generation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?