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

[ITG-103] List of relation types added to Task #38

Merged
merged 11 commits into from
May 11, 2022
Merged

Conversation

bkulawska
Copy link
Contributor

Done some code refactoring with Janek while implementing the task; there was a problem with loop imports - we resolved that, not sure if it's optimal :)

@bkulawska bkulawska requested a review from tgargula May 6, 2022 13:11
@bkulawska bkulawska changed the title [ITG-54] List of relation types added to Task [ITG-103] List of relation types added to Task May 6, 2022
Copy link
Contributor

@tgargula tgargula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comentillos

bode/bode/models/task.py Show resolved Hide resolved
bode/bode/models/task_actions.py Show resolved Hide resolved
bode/bode/models/task_actions.py Show resolved Hide resolved
bode/bode/models/task_relations_actions.py Outdated Show resolved Hide resolved
bode/bode/resources/tasks/schemas.py Outdated Show resolved Hide resolved
bode/bode/models/task_relations_actions.py Show resolved Hide resolved
bode/bode/models/task_relations_actions.py Outdated Show resolved Hide resolved
bode/bode/resources/tasks/schemas.py Outdated Show resolved Hide resolved
@bkulawska bkulawska requested a review from gregori0o May 11, 2022 08:55
parent 57aafeb
author bkulawska <kulawska1507@gmail.com> 1652259157 +0200
committer bkulawska <kulawska1507@gmail.com> 1652260859 +0200

Made suggested changes.

Fixed tags adding

[NOJIRA] feat: add github bot (#40)

* feat: add github bot

* fix: github bot run

* fix: another fix approach

* feat: add quotes

* feat: rename and fix indentation

* feat: improve order and fix yml syntax

* fix: excluded labels

* fix: plzkillmeicannotwriteinyaml

* fix: maybeorderchangewillhelp

* fix: dontworryillsquashitanyway

* fix: ifitisjustthisindentihateyaml

* fix: avadakedavra

* fix: maybedots?

Fixed import.

damniwillsquashitiswear
Copy link
Contributor

@gregori0o gregori0o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found mistake with mapping name of relation types. The same is in file task_relations/api.py.

bode/bode/models/task_relations_actions.py Outdated Show resolved Hide resolved
@bkulawska bkulawska requested a review from gregori0o May 11, 2022 21:22
Copy link
Contributor

@gregori0o gregori0o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💕💪💪

@bkulawska bkulawska merged commit af135fc into develop May 11, 2022
@bkulawska bkulawska deleted the features/ITG-103 branch May 11, 2022 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants