Skip to content

Switch to rxing for QR code detection#1635

Merged
FooIbar merged 1 commit intomainfrom
fb/rxing
Sep 16, 2024
Merged

Switch to rxing for QR code detection#1635
FooIbar merged 1 commit intomainfrom
fb/rxing

Conversation

@FooIbar
Copy link
Copy Markdown
Owner

@FooIbar FooIbar commented Sep 16, 2024

Maybe we can remove the tag requirement since performance improves?

@FooIbar FooIbar requested a review from revonateB0T September 16, 2024 09:45
@revonateB0T
Copy link
Copy Markdown
Collaborator

Maybe we can remove the tag requirement since performance improves?

We'd better not, just apply to new galleries as well.

@FooIbar
Copy link
Copy Markdown
Owner Author

FooIbar commented Sep 16, 2024

Maybe we can remove the tag requirement since performance improves?

We'd better not, just apply to new galleries as well.

Then we need to fill tags if remove ads is enabled

@FooIbar FooIbar merged commit 4bb6b0b into main Sep 16, 2024
@FooIbar FooIbar deleted the fb/rxing branch September 16, 2024 09:54
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