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

Add user_id to action_worker #2715

Closed
rrenkert opened this issue Nov 6, 2024 · 2 comments · Fixed by #2717
Closed

Add user_id to action_worker #2715

rrenkert opened this issue Nov 6, 2024 · 2 comments · Fixed by #2717
Assignees
Labels
enhancement General enhancement which is neither bug nor feature high

Comments

@rrenkert
Copy link
Member

rrenkert commented Nov 6, 2024

To have an association from the action_worker to a user, a new field user_id should be added to the action_worker. The field should always contain the request user id.

Adding the field needs a migration. Since action_workers are used temporarily we can simply delete all old action_workers. @peb-adr before we merge this migration we should talk about the impact.

@rrenkert rrenkert added high enhancement General enhancement which is neither bug nor feature labels Nov 6, 2024
@rrenkert
Copy link
Member Author

rrenkert commented Nov 6, 2024

Talked to @peb-adr: We already use the concept of a user with the id -1 and want to use -1 for old action_workers as user_id. So in the migration all action_workers are kept with the user_id -1.

@peb-adr
Copy link
Member

peb-adr commented Nov 6, 2024

We already use the concept of a user with the id -1

To be more specific, user_id -1 is set for positions that are created as a result of an internal action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement General enhancement which is neither bug nor feature high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants