-
-
Notifications
You must be signed in to change notification settings - Fork 11
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 Breakdown Prediction for Stats too Low #159
Comments
I feel this should be on the API side no? The API should not return a prediction breakdown for these people. I think that's how it was before no? If the breakdown map is null it won't add the box. |
There's a discussion to have it still send it on the API side, but have a yes/no toggle on the plugin side, so we're looking at that atm. |
We'd either need to have an extra field in the data sent to the plugin (I'm not a fan of having to look for a hard coded string 'stats too low') or have a parameter on the route to send the breakdown or not when that condition happens. |
Should be solved on API side |
We'd like to do the following:
In cases of Stats too low (ex. an account below 2m xp) we would remove the breakdown predictions, as they're not entirely relevant.
Let us know if you think this would be a good change ✅ , or to keep the current system ❌.
This can be a default off config for stats too low, and allow users to toggle it on -- if they'd like to.
The text was updated successfully, but these errors were encountered: