Closed sevein closed 2 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 53.07%. Comparing base (
cea3fe7
) to head (81b750c
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I've updated all pending dependencies used by the dashboard, excepting
@types/node
which we still want to mantain on v20.x because we haven't upgrade to Node.js 22 yet.For the record, the process I follow looks as follows:
npm run deps
(mentioned here) which launches an interactive user interface listing available updates,npm run build
) and test (npm run test
); confirm it works - if changes are required in our codebase, I launch vitest withnpx vitest dev
to watch the code and run the tests automatically while I make the necessary updates. Often I need to read the release notes, look up breaking changes, etc.npx prettier . --write
to fix.During the upgrade, I've had to dealt with the following issues:
vite.config.ts
) related to the use of mixed declarations in Bootstrap. I've seen that this is being addressed in Bootstrap v5.3.4 but it has not been released yet.UUID.test.ts
to mockdocument.execCommand('copy')
which is the fallback mechanism to interact with the clipboard (seeuseClipboard
for more details). I couldn't figure out how to fake the navigator permission instead - added a TODO note.Fixes #1021.