Feat/clin 2989 #539
Feat/clin 2989 #539
Conversation
Coverage report for
|
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 42e2fd9
Project Coverage and TestStatements : 33.62% ( 10580/31461 ) Test Suites: 58 passed, 58 total |
claudia1296
requested review from
emmanuelnau
and removed request for
dtremblay-sher
| const hasEmptyQuery = emptyQueries.length >= 1; | ||
| const getQueryIndexById = (id: string) => queriesState.queries.findIndex((obj) => obj.id === id); | ||
| const getQueryIndexById = (id: string) => queryStateQueriesWithoutFilter.findIndex((obj) => obj.id === id); | ||
| const canCombine = |
There was a problem hiding this comment.
Je sais pas si queryStateQueriesWithoutFilter est un gros tableau, mais c'est dommage de le parcourir 2 fois pour setter emptyQueries et canCombine. Je pense qu'il est possible de le parcourir une seule fois et setter les deux ;)
There was a problem hiding this comment.
Je pense même qu'il est possible de profiter de son iteration pour créé le query bar à la fin et éviter le map ^^
| if (!filterQueryToIgnore?.includes(((c as IValueFilter).content as IValueContent).field)) { | ||
| toRemove = true; | ||
| } | ||
| if (Array.isArray((c as IValueFilter).content)) { |
There was a problem hiding this comment.
Ici le toRemove = false est utile que si toRemove = true non ?
Du coup je pense qu'il peut être mis dans la condition
#FEAT : Flag Filter
Description
JIRA LINK
Acceptance Criterias
Validation
Screenshot
Before
After
QA