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(AffectedClustersTable): Reset filters button update #186

Merged

Conversation

Fewwy
Copy link
Collaborator

@Fewwy Fewwy commented Mar 21, 2022

The Reset filters button had the wrong name "Clear filters"

Had to update it and tests

@gkarat gkarat merged commit c5111cc into RedHatInsights:master Mar 21, 2022
gkarat pushed a commit that referenced this pull request Mar 21, 2022
## [1.0.15](v1.0.14...v1.0.15) (2022-03-21)

### Bug Fixes

* **AffectedClustersTable:** rename reset filters button ([#186](#186)) ([c5111cc](c5111cc))
@gkarat
Copy link
Collaborator

gkarat commented Mar 21, 2022

🎉 This PR is included in version 1.0.15 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

gkarat added a commit that referenced this pull request Mar 24, 2022
* fix(RecsListTable): Reset filters button behavior update (#183)

The recommendations list page Reset filters button change: the button is always available; if the button is clicked with the default filters, then the data are re-fetched; if the button is clicked with extra filters applied, it sets them to the default set.

* fix(ClusterListTable): Updated the reset filters behavior (#184)

The clusters list page Reset filters button change: the button is always available; if the button is clicked with the default filters, then the data are re-fetched; if the button is clicked with extra filters applied, it sets them to the default set.

* chore(release): 1.0.14 [skip ci]

## [1.0.14](v1.0.13...v1.0.14) (2022-03-21)

### Bug Fixes

* **ClusterListTable:** Updated the reset filters behavior ([#184](#184)) ([e184d33](e184d33))
* **RecsListTable:** Reset filters button behavior update ([#183](#183)) ([2287020](2287020))

* fix(AffectedClustersTable): rename reset filters button (#186)

* chore(release): 1.0.15 [skip ci]

## [1.0.15](v1.0.14...v1.0.15) (2022-03-21)

### Bug Fixes

* **AffectedClustersTable:** rename reset filters button ([#186](#186)) ([c5111cc](c5111cc))

* fix(disable modal): bug with multiple selections (#187)

This fixes a bug: if user selects multiple clusters, unchecks
"Disable recommendation for selected clusters" flag, then the
recommendation will only be disabled for selected clusters. This is incorrect, and
the recommendation should be disabled for all clusters in this scenario.

* chore(release): 1.0.16 [skip ci]

## [1.0.16](v1.0.15...v1.0.16) (2022-03-23)

### Bug Fixes

* **disable modal:** bug with multiple selections ([#187](#187)) ([659af06](659af06))

* chore(deps): bump axios from 0.26.0 to 0.26.1 (#172)

Bumps [axios](https://github.com/axios/axios) from 0.26.0 to 0.26.1.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/master/CHANGELOG.md)
- [Commits](axios/axios@v0.26.0...v0.26.1)

---
updated-dependencies:
- dependency-name: axios
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(tables): revert the time-based refresh feature

This reverts the feature of data time-based refresh (was set to 20
seconds) in the clusters list and recommendations list tables.

* chore: leave FIXMEs at the improper title change

* chore(release): 1.0.17 [skip ci]

## [1.0.17](v1.0.16...v1.0.17) (2022-03-24)

### Bug Fixes

* **tables:** revert the time-based refresh feature ([5961ef7](5961ef7))

Co-authored-by: Fewwy <62722417+Fewwy@users.noreply.github.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants