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

UISACQCOMP-228 Revert changes in useFilters hook #830

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

usavkov-epam
Copy link
Contributor

@usavkov-epam usavkov-epam commented Nov 12, 2024

@usavkov-epam usavkov-epam self-assigned this Nov 12, 2024
Copy link

github-actions bot commented Nov 12, 2024

Jest Unit Test Statistics

    1 files  ±0  219 suites  ±0   3m 58s ⏱️ -1s
578 tests ±0  576 ✔️ ±0  2 💤 ±0  0 ±0 
581 runs  ±0  579 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit 2ac8436. ± Comparison against base commit 676e061.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 12, 2024

BigTest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 2ac8436. ± Comparison against base commit 676e061.

♻️ This comment has been updated with latest results.

@usavkov-epam usavkov-epam requested review from a team and BogdanDenis November 12, 2024 14:25
Copy link

@vashjs vashjs left a comment

Choose a reason for hiding this comment

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

Hi @usavkov-epam I've approved, changes looks good to me.
But I think it would be nice to add ticket number in PR title to have links from jira + in description could you add PR where it was added initially, to have clear history.

@usavkov-epam usavkov-epam changed the title Revert changes in useFilters hook UISACQCOMP-228 Revert changes in useFilters hook Nov 12, 2024
@usavkov-epam usavkov-epam merged commit 7f664e2 into master Nov 12, 2024
6 checks passed
@usavkov-epam usavkov-epam deleted the UISACQCOMP-228 branch November 12, 2024 15:03
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.

3 participants