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

[#190] display responsible organ on the task page and add a tooltip f… #234

Merged
merged 5 commits into from
Aug 10, 2023

Conversation

cafischer
Copy link
Collaborator

closes #190

@cafischer cafischer marked this pull request as ready for review June 24, 2023 18:30
Copy link
Collaborator

@mdrie mdrie left a comment

Choose a reason for hiding this comment

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

I would merge this with my two suggested changes, but I am not sure, if you would agree.

cpmonitor/templates/task.html Outdated Show resolved Hide resolved
e2e_tests/database/test_database.json Outdated Show resolved Hide resolved
e2e_tests/database/test_database.json Outdated Show resolved Hide resolved
cpmonitor/models.py Outdated Show resolved Hide resolved
cpmonitor/models.py Outdated Show resolved Hide resolved
@mdrie
Copy link
Collaborator

mdrie commented Jun 27, 2023

When going through this, I also wonder, if we should do it with a dedicated model and a ManyToMany relation. @cafischer, @fblampe, @kleins05, @Holger-GZ what do you think?

@cafischer cafischer force-pushed the 190-responsible-organ branch 2 times, most recently from db9a4e3 to 10844dc Compare July 2, 2023 12:39
@cafischer
Copy link
Collaborator Author

For now I don't see the value of moving it into another model. What would be the arguments for doing that?

@cafischer cafischer force-pushed the 190-responsible-organ branch 2 times, most recently from ae605db to ad9e12d Compare July 2, 2023 13:15
@kleins05
Copy link
Collaborator

kleins05 commented Jul 3, 2023

With respect to having a separate model, I agree with @cafischer. From my point of view, as long as it does not avoid a lot of extra work in the future, we should never implement anything until one of the pilots/teams actually needs it.

@fblampe
Copy link
Collaborator

fblampe commented Aug 10, 2023

@mdrie's comments are now accounted for so I'll merge this.

@fblampe fblampe merged commit 56737ee into main Aug 10, 2023
1 check passed
@fblampe fblampe deleted the 190-responsible-organ branch August 10, 2023 16:19
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.

Add short name for responsible_organ and display in left sidebar
4 participants