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

Fix UserTask Status not being updated #50855

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

marcoandredinis
Copy link
Contributor

The Status field for UserTasks was not being correctly updated when the Spec.State was not changed.

@marcoandredinis marcoandredinis force-pushed the marco/fix_usertask_status_upsert branch 2 times, most recently from 85bc2f9 to c3ae45a Compare January 9, 2025 17:40
lib/auth/usertasks/usertasksv1/service.go Outdated Show resolved Hide resolved
@marcoandredinis marcoandredinis force-pushed the marco/fix_usertask_status_upsert branch from c3ae45a to 5a5498a Compare January 9, 2025 18:13
The Status field for UserTasks was not being correctly updated when the
Spec.State was not changed.
@marcoandredinis marcoandredinis force-pushed the marco/fix_usertask_status_upsert branch from 5a5498a to 8f0e59c Compare January 10, 2025 10:26
@marcoandredinis marcoandredinis force-pushed the marco/fix_usertask_status_upsert branch from 8f0e59c to 3d93237 Compare January 10, 2025 10:34
@marcoandredinis marcoandredinis added this pull request to the merge queue Jan 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 10, 2025
@marcoandredinis marcoandredinis added this pull request to the merge queue Jan 10, 2025
Merged via the queue into master with commit 66f82c2 Jan 10, 2025
41 checks passed
@marcoandredinis marcoandredinis deleted the marco/fix_usertask_status_upsert branch January 10, 2025 11:58
@public-teleport-github-review-bot

@marcoandredinis See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants