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 category search in explore #6093

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

parneet-guraya
Copy link
Contributor

@parneet-guraya parneet-guraya commented Dec 30, 2024

Fixes:
#6043
#5910

Tests performed (required)

Tested in prodDebug

Screenshots (for UI changes only)

Record_2024-12-31-02-15-42.mp4

@parneet-guraya
Copy link
Contributor Author

Everything works in prod variant but in beta only Items are loaded, is this expected? I saw the raw response for others which was empty {"batchcomplete":true} keeping it in draft if beta variant should also be working.

@nicolas-raoul your thoughts?

@nicolas-raoul
Copy link
Member

Eon't worry about beta, that fix is more urgent. Can you create a different issue about what's not working in beta?

Is it still a draft? 🙂

@parneet-guraya
Copy link
Contributor Author

Okay, I just kept it as the draft because of that, will mark it as ready now.

@parneet-guraya parneet-guraya marked this pull request as ready for review January 2, 2025 06:09
Signed-off-by: parneet-guraya <[email protected]>
@misaochan misaochan merged commit 038ae9a into commons-app:main Jan 3, 2025
1 check passed
@parneet-guraya parneet-guraya deleted the fix-search branch January 3, 2025 21:49
@sivaraam
Copy link
Member

@parneet-guraya Next time try to have the PR description in the following format (note the lack of new lines):

Fixes #6043 #5910

This will ensure the issues are closed automatically when the PR is merged. Thanks! 🙂

sivaraam added a commit that referenced this pull request Jan 15, 2025
This is a change that replicates the fix done via PR #6093.
This is based over v5.1.1 as the fix seems necessary for
the category search to work properly on the explore screen.
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.

4 participants