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

CM-334: change how benefit plan is filtered in picker #50

Merged
merged 3 commits into from
Oct 5, 2023

Conversation

jdolkowski
Copy link
Collaborator

Copy link
Contributor

@dborowiecki dborowiecki left a comment

Choose a reason for hiding this comment

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

@jdolkowski this change also contains alphabetical order feature which I think should be implemented on backend (or at least merged in separate PR). Can you fix it?

@jdolkowski
Copy link
Collaborator Author

I fixed it

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jdolkowski jdolkowski requested a review from dborowiecki October 5, 2023 08:52
@dborowiecki dborowiecki merged commit 007b4cb into develop Oct 5, 2023
5 checks passed
@dborowiecki dborowiecki deleted the feature/CM-334 branch October 5, 2023 19:39
dragos-dobre added a commit that referenced this pull request Oct 18, 2023
* CM-335: fetch alphabetically ordered benefit plans (#51)

Co-authored-by: Jan <j.dolkowski@soldevelo.com>

* CM-334: change how benefit plan is filtered in picker (#50)

* CM-334: change bf name filtering

* CM-334: add placeholder translation

---------

Co-authored-by: Jan <j.dolkowski@soldevelo.com>

* add-sonar-ci: add sonar files (#53)

Co-authored-by: Jan <j.dolkowski@soldevelo.com>

---------

Co-authored-by: Jan <jdolkowski@soldevelo.com>
Co-authored-by: Jan <j.dolkowski@soldevelo.com>
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