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

Remove baton_id from team model #145

Open
redfast00 opened this issue Apr 23, 2024 · 1 comment
Open

Remove baton_id from team model #145

redfast00 opened this issue Apr 23, 2024 · 1 comment

Comments

@redfast00
Copy link
Member

The database model for team contains a baton_id; but this shouldn't be used (it should be calculated based on batonswitchovers). We should be able to remove this without any consequences; if anything breaks, this is a bug (since we then know this doesn't use the switchovers).

This is an easier issue than #134; we don't need to remove baton ID's entirely, just in the team model, which nobody should be using anyways.

@redfast00
Copy link
Member Author

This is apparently also in use by monitoring; this edition, we should take care to always update the baton_id when we do a baton switchover; next edition, we should remove this from the database on the first codenight.

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

No branches or pull requests

1 participant