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

fix: Empty search list after a research by barcode #4587

Merged
merged 1 commit into from Mar 22, 2022

Conversation

g123k
Copy link
Contributor

@g123k g123k commented Mar 22, 2022

Will fix #4552

When a barcode is searched directly through the SearchView, the Search Activity is shown briefly… and then the Product Activity. But if we press the back button from the product, we will see the search activity with an infinite loader.
This page should have been finished before, which is now possible thanks to a dedicated listener

…h Activity is shown briefly… and then the Product Activity

But if we press the back button from the product, we will see the search activity with an infinite loader.
This page should have been finished before, which is not possible thanks to a dedicated listener
@g123k g123k requested a review from a team as a code owner March 22, 2022 12:11
@sonarcloud
Copy link

sonarcloud bot commented Mar 22, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@teolemon
Copy link
Member

thanks for the attention to detail 👌 @g123k merging and deploying

@teolemon teolemon merged commit 8b50123 into openfoodfacts:develop Mar 22, 2022
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.

Empty search screen
2 participants