Skip to content

Conversation

@piechnikk
Copy link
Collaborator

@piechnikk piechnikk commented Dec 3, 2025

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Added a search input to filter configurations, automatically updating the left-hand file tree view

@piechnikk piechnikk marked this pull request as ready for review December 4, 2025 07:59
@piechnikk piechnikk requested a review from graduta as a code owner December 4, 2025 07:59
Copy link
Collaborator

@Deaponn Deaponn left a comment

Choose a reason for hiding this comment

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

Code looks great, I just have one small question

it('should show no results message or empty list for non-matching query', async function () {
await page?.waitForSelector(SELECTORS.searchInput);

const randomString = 'non_existing_non_existing_non_existing_non_existing_non_existing';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Have you considered changing the name of this variable to something like nonExistingKey?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It does make sense, I will rename this variable

@piechnikk piechnikk self-assigned this Dec 4, 2025
Copy link
Collaborator

@sKrzysieK sKrzysieK left a comment

Choose a reason for hiding this comment

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

LGTM, great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants