Skip to content

Conversation

@Mateusz-Dobrzynski
Copy link
Member

This PR fixes a bug occurring when using GET /tournament as infrastructure admin.

@Mateusz-Dobrzynski Mateusz-Dobrzynski self-assigned this Aug 17, 2025
@Mateusz-Dobrzynski Mateusz-Dobrzynski added the bug Something isn't working label Aug 17, 2025
@Mateusz-Dobrzynski Mateusz-Dobrzynski linked an issue Aug 17, 2025 that may be closed by this pull request
@jakubmanczak
Copy link
Member

jakubmanczak commented Aug 20, 2025

Apart from the getting of debates instead of tournaments, the flow in the functions addressed in the PR looks fine to me, and I can't reproduce this bug.

I:

  • wiped old data,
  • pulled most recent master,
  • authenticated as infrastructure admin,
  • created a tournament as infradmin,
  • got the tournament data with GET /tournament as expected.

How are you getting it to occur?

@Mateusz-Dobrzynski Mateusz-Dobrzynski changed the title [69] Fix get_tournaments [69] Get teams instead of getting debates Aug 25, 2025
@Mateusz-Dobrzynski
Copy link
Member Author

Did the same and could not reproduce the bug. I must've made some wrong conclusions out of haste, sorry about that. Regardless, this PR fixes another bad function call, so I'm renaming it now.

Copy link
Member

@jakubmanczak jakubmanczak left a comment

Choose a reason for hiding this comment

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

@jakubmanczak jakubmanczak merged commit bbcd11d into master Aug 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get teams instead of getting debates

3 participants