-
Notifications
You must be signed in to change notification settings - Fork 130
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
IndexError: list index out of range #445
Comments
This is related to external feeds, i will take a look after project administrative tasks |
I find this bug to be introduced during @joelvai 's refactoring in december where the "none" option is commented out. a7015ed#diff-6a2e0d37f88f35d4bcc3e1d9da39a4b4 suggest default value to be |
Is there a planned fix for this bug? |
Ok, I'm able to reproduce the problem only by using a input value outside of proposed:
|
If allowed input is in range 1-5 and user entered 6 or more, treat this as 5. Closes: Codaone#445
I already made the fix on my branch here, the bug i did not find on whiptail but in the config elements. Here is the pull request: #503 Why are we making the assignment of this redundant? @PermieBTS @vvk |
@PermieBTS asked me to look into this bug. |
There was some confusion between octo and I over reassigning this issue to vvk. |
Expected Behavior
I assume config just moves onto next step.
Actual Behavior
Config outputs an error.
Steps to Reproduce the Problem
Specifications
The text was updated successfully, but these errors were encountered: