Skip to content

feat: [IOCOM-3243] FP-TS removal from messages' reducers#8234

Open
forrest57 wants to merge 1 commit into
masterfrom
IOCOM-3243-fe-rimozione-fp-ts-da-reducer-messaggi
Open

feat: [IOCOM-3243] FP-TS removal from messages' reducers#8234
forrest57 wants to merge 1 commit into
masterfrom
IOCOM-3243-fe-rimozione-fp-ts-da-reducer-messaggi

Conversation

@forrest57

Copy link
Copy Markdown
Contributor

Short description

this PR removes the remaining references of FP-TS from the messages' reducers

List of changes proposed in this pull request

  • refactored various reducers to remove FP-TS and increase readability
  • updated tests

How to test

automated tests should pass. also, using the dev-server, verify all kinds of standard and thirdParty messages work as expected.

@forrest57 forrest57 self-assigned this Jul 3, 2026
@forrest57 forrest57 added the IO-FIP IO Foundation - Interazioni & Preferenze label Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Jira Pull Request Link

This Pull Request refers to Jira issues:

@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.92%. Comparing base (590562f) to head (c427536).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8234      +/-   ##
==========================================
+ Coverage   62.77%   62.92%   +0.14%     
==========================================
  Files        1952     1948       -4     
  Lines       37085    37022      -63     
  Branches     6206     6207       +1     
==========================================
+ Hits        23280    23295      +15     
+ Misses      12420    12344      -76     
+ Partials     1385     1383       -2     
Files with missing lines Coverage Δ
...onents/MessageDetail/MessageDetailsAttachments.tsx 100.00% <100.00%> (ø)
...p/ts/features/messages/store/reducers/archiving.ts 100.00% <100.00%> (+56.81%) ⬆️
...ts/features/messages/store/reducers/detailsById.ts 86.20% <100.00%> (+0.49%) ⬆️
...p/ts/features/messages/store/reducers/downloads.ts 100.00% <100.00%> (ø)
...pp/ts/features/messages/store/reducers/payments.ts 97.26% <100.00%> (-0.08%) ⬇️
...features/messages/store/reducers/thirdPartyById.ts 97.29% <100.00%> (+13.51%) ⬆️
...n-app/ts/features/messages/types/thirdPartyById.ts 100.00% <ø> (ø)
.../main-app/ts/features/pn/components/F24Section.tsx 100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 590562f...c427536. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

IO-FIP IO Foundation - Interazioni & Preferenze

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant