Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 26, 2025

What do these changes do?

reported by @ahalder2

This PR fixes issues related to filtering behavior in the StudyBrowser, specifically addressing problems when filters are applied but should not display results.

Fix

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Sep 26, 2025
@odeimaiz odeimaiz requested a review from Copilot September 26, 2025 11:04
@odeimaiz odeimaiz added this to the Cheops milestone Sep 26, 2025
@odeimaiz odeimaiz added bug buggy, it does not work as expected a:frontend issue affecting the front-end (area group) labels Sep 26, 2025
@odeimaiz odeimaiz marked this pull request as ready for review September 26, 2025 11:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes issues related to filtering behavior in the StudyBrowser component, specifically addressing problems when filters are applied but should not display results.

  • Prevents workspaces and folders from being displayed when certain filters are enabled
  • Improves parameter handling for text-based search queries
  • Adds conditional logic to skip result processing when filters are active

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link

@odeimaiz odeimaiz changed the title 🐛 [Frontend] Fix 🐛 [Frontend] Fix: filter tags with empty text Sep 26, 2025
@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2025

🧪 CI Insights

Here's what we observed from your CI run for 10cfb8f.

✅ Passed Jobs With Interesting Signals

Pipeline Job Signal Health on master Retries 🔍 CI Insights 📄 Logs
CI system-tests Base branch is broken, but the job passed. Looks like this might be a real fix 💪 Broken 0 View View

@odeimaiz odeimaiz merged commit 3d1e310 into ITISFoundation:master Sep 26, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants