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

Deactivated contests still show up in API #2688

Open
meisterT opened this issue Sep 16, 2024 · 4 comments
Open

Deactivated contests still show up in API #2688

meisterT opened this issue Sep 16, 2024 · 4 comments
Labels

Comments

@meisterT
Copy link
Member

After we deactivate a contest, it is gone from dropdowns etc. in the web interface, but it is still available in the API. I would argue, it should also disappear in the API after deactivation.

@meisterT meisterT added the API label Sep 16, 2024
@as6325400
Copy link
Contributor

Can I try to handle it? Should I do it in the latest main branch? Thank you.

@Kevinjil
Copy link
Contributor

Isn't this already implemented:

@meisterT
Copy link
Member Author

Good find Kevin! This has been added by @thijskh in 5fd1413 originally.

My personal opinion is that we should be consistent between UI and API and change the default to filter out deactivated contests.

WDYT?

@vmcj
Copy link
Member

vmcj commented Sep 28, 2024

Good find Kevin! This has been added by @thijskh in 5fd1413 originally.

My personal opinion is that we should be consistent between UI and API and change the default to filter out deactivated contests.

WDYT?

So to add a flag alsoInactive and default only show active contests? Makes sense to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants