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

Gui update #5

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

Gui update #5

wants to merge 3 commits into from

Conversation

Cerepol
Copy link

@Cerepol Cerepol commented Aug 7, 2024

Adds gui and fixes an issue where it did not mark pages if they weren't already populated

@SenorSmartyPants
Copy link
Owner

I'll be happy to merge this. But could you delete the commented out code you have in sections of code you added.

@Cerepol
Copy link
Author

Cerepol commented Aug 8, 2024

Sure, updated to remove those comments. Thanks for making this btw! its great for my collection

@SenorSmartyPants
Copy link
Owner

SenorSmartyPants commented Aug 29, 2024

I finally pulled your branch and tried it locally. I get an error, here's the debug console output

Calling 'MarkScannerPage'...
unsupported operand type(s) for +=: 'bool' and 'instancemethod'

Looks like this error was introduced on the code clean up commit

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.

2 participants