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.

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

Upgrade/element web v1.11.44 #763

Merged
merged 388 commits into from
Nov 2, 2023

Conversation

estellecomment
Copy link
Contributor

@estellecomment estellecomment commented Oct 9, 2023

Fixes #734

TODO

  • fix conflicts
  • patches
    • better-help-settings-2 : reimplement it's too complicated to understand.
    • registration-for-mainlining : not sure what happened, probly start over
  • ci
    • linter
    • tests : enzyme error. Is it time to remove enzyme like reactsdk ?
    • cypress : settings.spec.ts is broken by translation problem.

Extra things to think about and do

  • translations are frozen, halfway through transitioning to another translation system. Are they all broken ? Partly. Transformation en module de la traduction de l'app #703
  • translations now are organized into nested categories which is nice. Should we do this too ? Now or later ? -> later. Too much stuff already
  • manual tests
  • checkboxes in the issue
  • couleur des avatars pétée, c'est moins lisible qu'avant -> filed Colors of avatars are broken (after upgrade 1.11.14) #772, will solve after this PR
  • all the languages came back, delete them. They populate the dropdown for chosing the language, it's too noisy. -> added a patch to filter out languages. That way we leave the language files alone.

renovate bot and others added 30 commits June 27, 2023 12:26
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <7t3chguy@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update all non-major dependencies

* Roll back typescript

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <7t3chguy@gmail.com>
Bumps [stylelint](https://github.com/stylelint/stylelint) from 15.9.0 to 15.10.1.
- [Release notes](https://github.com/stylelint/stylelint/releases)
- [Changelog](https://github.com/stylelint/stylelint/blob/main/CHANGELOG.md)
- [Commits](stylelint/stylelint@15.9.0...15.10.1)

---
updated-dependencies:
- dependency-name: stylelint
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <7t3chguy@gmail.com>
@estellecomment
Copy link
Contributor Author

estellecomment commented Oct 9, 2023

Petite regression du dialog de /rageshake. (est-ce que ca vaut le coup de la fixer ? Ajouté dans la checklist)

Expected :

image

Obtained :

image

@estellecomment
Copy link
Contributor Author

estellecomment commented Oct 9, 2023

couleur des avatars pétée, c'est moins lisible qu'avant. (ajouté à la checklist dans la PR description)

Expected :

image

Obtained :

image

Edit : this is filed already in element-web : element-hq/element-web#26285, on 13 sep. No news.

@estellecomment
Copy link
Contributor Author

estellecomment commented Oct 9, 2023

  • translations : key format has changed, so many translations are broken in our customisations files.

-> filed #769, will fix after this PR is merged.
Edit : actually now its #703

@estellecomment
Copy link
Contributor Author

Translation situation : translation keys are in the process of changing format : "Secure Backup" -> "common|secure_backup". Some keys are already migrated, some are not.

The new format is not custom-translateable with our current setup (tchap-translations.json doesn't work). So for 1.11.44 part of our custom translations disappear. For the next upgrade they will probably all disappear !

Issue already filed, we need to make a module : #703

.github/workflows/build_debian.yaml Show resolved Hide resolved
.github/workflows/dockerhub.yaml Show resolved Hide resolved
+ // :tchap: return _t("Continue");
+ return _t("Activate");
return _t("action|continue");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be Activate?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another good catch !

Ma conclusion c'est qu'on devrait e2e tester les custom translations 🤔
On pourrait utiliser des snapshots. J'aime pas trop les snapshots parce qu'ils cassent dès que le moindre bout de html change, ce qui arrive hors-bug.
Sinon on code des tests de translations... Ou une sorte de snapshots flexibles, je sais pas si ca existe... 🤔

Filed #785

@estellecomment estellecomment merged commit 47f5c81 into develop_tchap Nov 2, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Priority 1 upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to element-web 1.11.44