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

negative review count for custom queue builder #10

Closed
lovac42 opened this issue Jan 6, 2019 · 3 comments
Closed

negative review count for custom queue builder #10

lovac42 opened this issue Jan 6, 2019 · 3 comments
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@lovac42
Copy link
Owner

lovac42 commented Jan 6, 2019

When using the custom queue builder, there is an issue with miscounts when the parent deck contains subdecks within subdecks and more subdecks within those subdecks. Currently, it does not respect the limits imposed by the middle subdecks, only the selected parent and grandchild.

This can be avoided by not setting a limit on mid level decks (e.g. 9999). The actual fix will require checking the limit on sibling decks as the queue is built and offsetting the values from the total limit.

@lovac42 lovac42 added bug Something isn't working wontfix This will not be worked on labels Jan 6, 2019
@cjdduarte
Copy link

Thanks, one last question, if I select one of these two options, what addon will prioritize (sort) first?
Sorry, my English is not very good, and in the translation, I was a bit confused.

image

@lovac42 lovac42 pinned this issue Jan 25, 2019
@lovac42
Copy link
Owner Author

lovac42 commented Jan 25, 2019

You posted in the wrong place.
Each addon is separate, their configs has nothing to do with one another.

@lovac42 lovac42 closed this as completed Jan 25, 2019
@cjdduarte
Copy link

OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants