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.

Sign up for GitHub

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

Jump to bottom

Entity comment column not updated when only the comment is changed #9134

Closed
fniessink opened this issue Jul 3, 2024 · 0 comments · Fixed by #9170
Closed

Entity comment column not updated when only the comment is changed #9134

fniessink opened this issue Jul 3, 2024 · 0 comments · Fixed by #9170
Assignees
@fniessink
Labels
Bug Something isn't working

Comments

@fniessink
Copy link
Member

Describe the bug
Entity comment column not updated when only the comment is changed. When the entity status is updated the column is updated as well.

To Reproduce
Steps to reproduce the behaviour:

  1. Go to a measurement entity with status unconfirmed.
  2. Add a comment
  3. Notice that the comment in the comment column is not updated
  4. Chang the status to confirmed
  5. Notice that the comment in the comment column is updated

Expected behaviour
Comment in the comment column should be updated without changing the measurement entity status.

Reported by @Sebastiaan127001
@fniessink fniessink added the Bug Something isn't working label Jul 3, 2024
@fniessink fniessink moved this from Inbox to Development in progress in Quality-time backlog Jul 3, 2024
@fniessink fniessink moved this from Development in progress to Ready in Quality-time backlog Jul 3, 2024
@fniessink fniessink moved this from Ready to Development in progress in Quality-time backlog Jul 4, 2024
@fniessink fniessink self-assigned this Jul 4, 2024
fniessink added a commit that referenced this issue Jul 4, 2024
@fniessink
Fix missing measurement entity status rationale.
8a759db 
Don't hide the measurement entity status rationale when the status of the measurement entity is "Unconfirmed".

Fixes #9134.
@fniessink fniessink mentioned this issue Jul 4, 2024
Merged
@fniessink fniessink linked a pull request Jul 4, 2024 that will close this issue
Fix missing measurement entity status rationale. #9170
Merged
@fniessink fniessink moved this from Development in progress to Reviewing in progress in Quality-time backlog Jul 4, 2024
@github-project-automation github-project-automation bot moved this from Reviewing in progress to Merged in Quality-time backlog Jul 4, 2024
@fniessink fniessink moved this from Merged to Release candidate released in Quality-time backlog Jul 5, 2024
@fniessink fniessink moved this from Release candidate released to Released in Quality-time backlog Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fniessink fniessink

Labels
Bug Something isn't working
Projects
Quality-time backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix missing measurement entity status rationale. ICTU/quality-time
1 participant
@fniessink