Labels Milestones

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

Merged

ashette merged 32 commits from 79-students-tags-generation into dev 5 years ago

Conversation 26 Commits 32 Files Changed 32

ashette commented 5 years ago (Migrated from gitlab.com)

Closes #79

Closes #79

ashette commented 5 years ago (Migrated from gitlab.com)

changed milestone to %7

changed milestone to %7

ashette commented 5 years ago (Migrated from gitlab.com)

added 1 commit

• b4c11a7f - #79 year generation

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 5 years ago (Migrated from gitlab.com)

added 1 commit

• 7ce8b208 - #79 fixing bug

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 5 years ago (Migrated from gitlab.com)

added 1 commit

• 65dea9bc - #79 adding task planer generation

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 5 years ago (Migrated from gitlab.com)

added 1 commit

• 0de34c89 - #79 fixing planner and year generation

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 5 years ago (Migrated from gitlab.com)

added 1 commit

• c59bb45c - #79 fixing codestyle

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 5 years ago (Migrated from gitlab.com)

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 5 years ago (Migrated from gitlab.com)

unmarked as a Work In Progress

unmarked as a **Work In Progress**

ashette commented 5 years ago (Migrated from gitlab.com)

assigned to @romanov73

assigned to @romanov73

romanov73 commented 5 years ago (Migrated from gitlab.com)

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

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

romanov73 commented 5 years ago (Migrated from gitlab.com)

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

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

romanov73 commented 5 years ago (Migrated from gitlab.com)

param?

param?

romanov73 commented 5 years ago (Migrated from gitlab.com)

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

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

romanov73 commented 5 years ago (Migrated from gitlab.com)

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

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

ashette commented 5 years ago (Migrated from gitlab.com)

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

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

ashette commented 5 years ago (Migrated from gitlab.com)

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

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

romanov73 commented 5 years ago (Migrated from gitlab.com)

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

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

ashette commented 5 years ago (Migrated from gitlab.com)

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 5 years ago (Migrated from gitlab.com)

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 5 years ago (Migrated from gitlab.com)

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 5 years ago (Migrated from gitlab.com)

added 1 commit

• bc9cca62 - #79 refactoring

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 5 years ago (Migrated from gitlab.com)

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 5 years ago (Migrated from gitlab.com)

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

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

romanov73 commented 5 years ago (Migrated from gitlab.com)

resolved all discussions

resolved all discussions

romanov73 commented 5 years ago (Migrated from gitlab.com)

merged

merged

romanov73 commented 5 years ago (Migrated from gitlab.com)

mentioned in commit 19a6b45cf9

mentioned in commit 19a6b45cf9e0841bb7a97b1905a58effbde1bf7f

The pull request has been merged as 19a6b45cf9.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b 79-students-tags-generation dev

git pull origin 79-students-tags-generation

Step 2:

Merge the changes and update on Gitea.

git checkout dev

git merge --no-ff 79-students-tags-generation

git push origin dev

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels   

backend

  

bug

  

frontend

  

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

  

выполняется

  

конференции

  

события

  

статьи

No Label

backend

bug

frontend

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

выполняется

конференции

события

статьи

Milestone

Set milestone

Clear milestone

No items

No Milestone

Работа со студентами

Projects

Set Project

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

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

Write Preview

Loading…

Cancel

Save

There is no content yet.