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

Support arbitrary code lengths. Fixes #11. #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JackZielke
Copy link

Support code lengths other than 6.

When adding codes, the default is 6 digits unless otherwise specified. Max length is 10.* This bumps the version since the config page is incompatible with the old version. The new code needs the code lengths and the new config page provides them. I have not tested the old app with the new config page. I expect the old code to ignore the lengths and still work.

PR #6 hardcoded the name and reduced the font size for all codes. This version reduces the font when needed and provides an optional number of digits on the config page.

  • 2^32 = 4294967296. The first digit is 0-4 so no one would really use 10 digits. This should cover any use.

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 this pull request may close these issues.

1 participant