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

Admin Form Schema Improvements #48

Open
tydolla00 opened this issue Mar 6, 2025 · 0 comments · May be fixed by #50
Open

Admin Form Schema Improvements #48

tydolla00 opened this issue Mar 6, 2025 · 0 comments · May be fixed by #50
Assignees

Comments

@tydolla00
Copy link
Owner

Note

These are just suggestions, it's up for discussion whether we want to take the time to implement it or not.

I'm thinking we can improve our form schema and be more explicit about the input. With future games in mind and us already knowing what we want to track for each, we can have the validation schema drive our ui.

For example, for Rocket League, validating that the winning team has more goals than the losing team. We know the point system, we could validate that the points are at least what the point system indicates. etc..

In addition, I would like to see if we can automate some things.

For example, right now we're able to automate the match winners but we should be able to keep track of the set winners as well. We could add additional info asking about the structure of the match. I'm not a big fan of this approach because sometimes they'll do a set as a bo3 vs a bo5 in the same video. But I think having a default as a backup for each game could be useful.

So for rocket league we'll assume that a set is a bo5 every time, if our winners win three matches straight we automatically set the set winners and close the accordion. They can still go and modify that set if needed.

@tydolla00 tydolla00 self-assigned this Mar 6, 2025
@tydolla00 tydolla00 linked a pull request Mar 8, 2025 that will close this issue
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

Successfully merging a pull request may close this issue.

1 participant