-
Notifications
You must be signed in to change notification settings - Fork 336
🐛(frontend) Collaboration upgrade #684
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
Conversation
70a689b
to
ff5b36e
Compare
CHANGELOG.md
Outdated
[unreleased]: https://github.com/numerique-gouv/impress/compare/v2.4.0...main | ||
[v2.4.0]: https://github.com/numerique-gouv/impress/releases/v2.4.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should revert this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yes
78c1f6e
to
7a3046b
Compare
src/frontend/package.json
Outdated
@@ -40,6 +40,7 @@ | |||
"@typescript-eslint/parser": "8.23.0", | |||
"docx": "9.1.1", | |||
"eslint": "8.57.0", | |||
"prosemirror-view": "1.37.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have to set a rule in renovate configuration ? Like ignoring it because we want to manage the version our self ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could but Blocknote made a fix for these 2 already:
- fix: compatibility with new react-pdf TypeCellOS/BlockNote#1494
- fix: upgrade prosemirror view and use correct clipboard import TypeCellOS/BlockNote#1431
Docx seems to work with newer version as well.
I have already a PR opened about bumping Blocknote (#640), I think it will be ready before renovate tried to bump them.
We upgrade blocknote to 0.23.2-hotfix.0, it includes a fix with the collaboration.
Blocknote does not pinned the version. We get bumped version instead of the version we want. We pin the version of blocknote globally to avoid this issue.
7a3046b
to
8f3cd20
Compare
Purpose
Last version of Blocknote (0.23.2) has a bug with the collaboration.
We upgrade Blocknote to
0.23.2-hotfix.0
that fix this problem.Proposal