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.

Sign up for GitHub

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

✨ table-controls: factor out duplicated code via new usePersistentState hook + misc cleanup + add initial documentation (DOCS.md and JSDoc comments) #1355

Merged
merged 102 commits into from
Oct 24, 2023

Proofread the rest of DOCS.md

06546cb
Select commit
Loading
Failed to load commit list.
Merged

✨ table-controls: factor out duplicated code via new usePersistentState hook + misc cleanup + add initial documentation (DOCS.md and JSDoc comments) #1355

Proofread the rest of DOCS.md
06546cb
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 23, 2023 in 0s

19.70% of diff hit (target 40.72%)

View this Pull Request on Codecov

19.70% of diff hit (target 40.72%)

Annotations

Check warning on line 63 in client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts#L63 

Added line #L63 was not covered by tests

Check warning on line 66 in client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts#L66 

Added line #L66 was not covered by tests

Check warning on line 68 in client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts#L68 

Added line #L68 was not covered by tests

Check warning on line 71 in client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/getActiveItemDerivedState.ts#L70-L71 

Added lines #L70 - L71 were not covered by tests

Check warning on line 36 in client/src/app/hooks/table-controls/active-item/useActiveItemEffects.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemEffects.ts#L35-L36 

Added lines #L35 - L36 were not covered by tests

Check warning on line 38 in client/src/app/hooks/table-controls/active-item/useActiveItemEffects.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemEffects.ts#L38 

Added line #L38 was not covered by tests

Check warning on line 42 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L41-L42 

Added lines #L41 - L42 were not covered by tests

Check warning on line 44 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L44 

Added line #L44 was not covered by tests

Check warning on line 46 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L46 

Added line #L46 was not covered by tests

Check warning on line 51 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L51 

Added line #L51 was not covered by tests

Check warning on line 55 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L55 

Added line #L55 was not covered by tests

Check warning on line 59 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L59 

Added line #L59 was not covered by tests

Check warning on line 63 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L61-L63 

Added lines #L61 - L63 were not covered by tests

Check warning on line 68 in client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemPropHelpers.ts#L68 

Added line #L68 was not covered by tests

Check warning on line 50 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L48-L50 

Added lines #L48 - L50 were not covered by tests

Check warning on line 54 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L54 

Added line #L54 was not covered by tests

Check warning on line 66 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L66 

Added line #L66 was not covered by tests

Check warning on line 69 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L69 

Added line #L69 was not covered by tests

Check warning on line 72 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L72 

Added line #L72 was not covered by tests

Check warning on line 75 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L75 

Added line #L75 was not covered by tests

Check warning on line 79 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L79 

Added line #L79 was not covered by tests

Check warning on line 81 in client/src/app/hooks/table-controls/active-item/useActiveItemState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/active-item/useActiveItemState.ts#L81 

Added line #L81 was not covered by tests

Check warning on line 60 in client/src/app/hooks/table-controls/expansion/getExpansionDerivedState.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/expansion/getExpansionDerivedState.ts#L60 

Added line #L60 was not covered by tests

Check warning on line 59 in client/src/app/hooks/table-controls/expansion/useExpansionPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/expansion/useExpansionPropHelpers.ts#L59 

Added line #L59 was not covered by tests

Check warning on line 66 in client/src/app/hooks/table-controls/expansion/useExpansionPropHelpers.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/src/app/hooks/table-controls/expansion/useExpansionPropHelpers.ts#L66 

Added line #L66 was not covered by tests
View more details on Codecov