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: CLIN-2916 fix saving custom filter after adding saved custom pill to it #503

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

meek0
Copy link
Contributor

@meek0 meek0 commented Aug 7, 2024

FIX

  • closes CLIN-2916

Description

JIRA LINK

Acceptance Criterias

Validation

  • Storybook add or modified
  • version Update in package.json and Release.md
  • Code Approved
  • QA Done
  • Design/UI Approved from design

@meek0 meek0 force-pushed the fix/clin-2916 branch 2 times, most recently from 3e3b02b to 654732e Compare August 7, 2024 13:31
Copy link

github-actions bot commented Aug 7, 2024

Coverage report for packages/ui

St.
Category Percentage Covered / Total
🟢 Statements 100% 0/0
🟢 Branches 100% 0/0
🟢 Functions 100% 0/0
🟢 Lines 100% 0/0

Test suite run success

0 tests passing in 0 suite.

Report generated by 🧪jest coverage report action from f7f936f

Copy link

github-actions bot commented Aug 7, 2024

Project Coverage and Test

Statements : 35.58% ( 10446/29358 )
Branches : 55.41% ( 599/1081 )
Functions : 32.65% ( 193/591 )
Lines : 35.58% ( 10446/29358 )

Test Suites: 56 passed, 56 total
Tests: 248 passed, 248 total
Snapshots: 0 total
Time: 85.783 s
Ran all test suites.

@meek0 meek0 force-pushed the fix/clin-2916 branch 4 times, most recently from 8067dcd to d00bd7e Compare August 7, 2024 18:35
@meek0 meek0 marked this pull request as ready for review August 9, 2024 15:43
@@ -1,6 +1,6 @@
{
"name": "@ferlab/ui",
"version": "10.4.6",
"version": "10.4.7-rc1",
Copy link
Contributor

Choose a reason for hiding this comment

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

je pense que tu as pas fait le npm install car le packag-lock.json devrait aussi avoir la version mise à jour

Copy link
Contributor Author

Choose a reason for hiding this comment

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

en effet, fixing it now

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

Successfully merging this pull request may close these issues.

2 participants