Add MCS

[weeman1337]

---

This change is marked as an internal change (Task), so will not be included in the changelog.

[jdauphant]

Solution to not having to copy manually the original files :

• Find a way to reference original files from the react sdk
• Create symlinks to original files
• Having a pre-compile script that auto-copy the original files before compilation

And if can't : we can add a CI check that Original is always the same.

[jdauphant]

Solution to not having to copy manually the original files :

• Find a way to reference original files from the react sdk
• Create symlinks to original files
• Having a pre-compile script that auto-copy the original files before compilation

And if can't : we can add a CI check that Original is always the same.

Another solution have been suggested by the beta.gouv.fr communauty : https://github.com/ds300/patch-package

[estellecomment]

When there is infected content, we should help the user understand what they can or can't do. They might get scared that the content will infect their machine for example. I think we need a bit more help text. For example in a dialog that opens when you try to download, and also when you hover on the red bug icon.
Has this feature been user-tested ?
All that can be done in a separate PR though, it's an enhancement, this version does the job.

[estellecomment]

can you specify the version of react-sdk, for tracking purposes ?
Currently tchap-web-v4 uses react-sdk 3.46.0, which has a slightly different version of this file.

[weeman1337]

@estellecomment can you check whether 1042876 works for you?

This is a quite creative solution based on your comments about using customisations.json to point to node_modules and Juliens hint to patch-package.

[estellecomment]

AND THIS IS DONNNNNNNE
Congrats everyone, I would like to thank my parents without whom I would not have made it so far

(two small comments)

[estellecomment]

Would be nice to reuse colors, like --var(warning) or something.
Not a blocker, if it's not a 5 minute task then let's file an issue instead

[weeman1337]

Totally agree. Unfortunately, when not using a custom theme we don't have this variables. This is a > 5 minutes task.

[jdauphant]

WIth Angelo, we want to use a different way to custom the theme : #130
It could be add to the issue when it will be priorized.

[jdauphant]

Are we good to merge this PR ?

[estellecomment]

Yes, good for me, @weeman1337 are you still working on some things ?