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

Jts/verji develop final sync to matrix-react-sdk-v3.109.0 #100

Open
wants to merge 444 commits into
base: verji-develop
Choose a base branch
from

Conversation

JohnSimonsen
Copy link

@JohnSimonsen JohnSimonsen commented Nov 27, 2024

Synd Verji-Develop to matrix-react-sdk v3.109.0

MUST:
searchbar.tsx --> deleted
- UIFeature.SearchInAllRooms: Needs Re-implementation
EmailAddresses.tsx --> deleted
- UIFeature.EmailAddressShowRemoveButton: Needs Re-implementation
- UIFeature.EmailAddressShowAddButton: Needs Re-Implementation
PhoneNumbers.tsx --> Deleted
- UIFeature.PhoneNumerShowRemoveButton: Needs Re-Implementation
- UIFeature.PhoneNumerShowAddButton: Needs Re-implementation
GeneralUserSettingsTab.tsx
- UIFeature.UserSettingsExternalAccount: Needs Re-Implementation
- UIFeatureUserSettingsChangePassword: Needs Re-Implementation
- UIFeature.UserSettingsSetIdServer: Needs Re-Implementation
- UIFeature.UserSettingsDiscovery: Needs Re-Implementation
- UIFeature.UserSettingsIntegrationManager: Needs Re-Implementation

RiotRobot and others added 30 commits July 9, 2024 13:18
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…-org#12750)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update dependency @sentry/browser to v8.15.0

* Fix type import

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…-org#12758)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update all non-major dependencies

* Iterate

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Split up slow Playwright tests

To optimise parallelism

Deals with:

```
Slow test file: read-receipts/redactions.spec.ts (5.4m)
Slow test file: read-receipts/new-messages.spec.ts (3.9m)
Slow test file: read-receipts/high-level.spec.ts (3.6m)
Slow test file: read-receipts/editing-messages.spec.ts (3.1m)
Slow test file: read-receipts/reactions.spec.ts (2.2m)
Slow test file: crypto/crypto.spec.ts (2.4m)
Slow test file: settings/appearance-user-settings-tab/appearance-user-settings-tab.spec.ts (1.2m)
Slow test file: composer/composer.spec.ts (1.1m)
Slow test file: crypto/verification.spec.ts (1.1m)
```

Signed-off-by: Michael Telatynski <[email protected]>

* Move around snapshots

Signed-off-by: Michael Telatynski <[email protected]>

* Fix test

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
…atrix-org#12755)

* Remove refs to deprecated method

`MatrixEvent.isEncryptedDisabledForUnverifiedDevices` is now deprecated

* Report new posthog code for withheld message keys
* Move integrations switch

This is the last thing to move out of 'general' now.

* unused import

* Move tests out to the SetIntegrationManager component

* Only a decade out

* Move playwright test to the new tab

* Update snapshot

* Update other snapshot
…atrix-org#12769)

* Use multiple workers in Playwright CI to make use of multiple cores

Signed-off-by: Michael Telatynski <[email protected]>

* Iterate

Signed-off-by: Michael Telatynski <[email protected]>

* Iterate

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
…atrix-org#12770)

* Hide tooltips for certain playwright screenshots to avoid flakiness

Signed-off-by: Michael Telatynski <[email protected]>

* Typo

Signed-off-by: Michael Telatynski <[email protected]>

* tsc

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
* Update dependency @vector-im/compound-web to v5.4.0

* Update snapshots

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>
* remove "Manually verify all remote sessions" setting

And remove all of the advanced encryption settings section, since that was the
only setting in there.

* yarn i18n

* also remove now-unused OrderedMultiController and PushToMatrixClientController
* Remove call to no-op `checkOwnCrossSigningTrust`

this is a no-op on rust crypto

* inline `SecurityManager.isCachingAllowed`

Since matrix-org#4789, this has just
been an obscure way to write a test of a local variable.

* Remove unused `CreateSecretStorageOpts.getKeyBackupPassphrase` parameter

This is unused on rust crypto (cf matrix-org/matrix-js-sdk#4313)
* Add logging to `getSecretStorageKey`

* Replace call to deprecated MatrixClient.hasSecretStorageKey

* Add/improve logging in `accessSecretStorage`

* Add/improve logging in SetupEncryptionStore.usePassPhrase
… 1:1 and Widget (matrix-org#12760)

* Align `widget_build_url_ignore_dm` with call behaviour switch between 1:1 and Widget

Signed-off-by: Michael Telatynski <[email protected]>

* Add tests

Signed-off-by: Michael Telatynski <[email protected]>

* Improve coverage

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
RiotRobot and others added 30 commits August 27, 2024 12:39
* Install deja-vu font

* Add comment
…atrix-org#12934)

* Improve PinningUtils.ts doc and use common methods to check pin or unpin.
Removed unused methods.

* Send room account data and state event in parallel

* Rerender MessageActionBar.tsx if there is a room pinned event

* Update pinning util tests

* Add test for room pinned events in MessageActionBar-test.tsx
* Fix reply message truncation on 2 lines

* Add e2e tests for reply
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…12917)

* Move pinned message hooks to a dedicated file

* Add a banner at the top of a room to display the pinned messages

* Put the pinning banner behind labs pinning labs flag

* Add redacted event support

* Handle UTD in pinning message banner

* Add tests for redaction

* Make all the banner clickable

* Add tests for PinnedMessageBanner.tsx

* Add e2e tests for the pinned message banner

* Review changes
* Implement download_file in widget driver

Signed-off-by: Michael Weimann <[email protected]>

* Fix test URIs

Signed-off-by: Michael Weimann <[email protected]>

* Use download-file branch as widget-api source

Signed-off-by: Michael Weimann <[email protected]>

* bump matrix-widget-api to 1.9.0

Signed-off-by: Kim Brose <[email protected]>

* prettier

Signed-off-by: Kim Brose <[email protected]>

---------

Signed-off-by: Michael Weimann <[email protected]>
Signed-off-by: Kim Brose <[email protected]>
Co-authored-by: Kim Brose <[email protected]>
* Allow user to set timezone

* Update test snapshots

---------

Co-authored-by: Florian Duros <[email protected]>
* Fix when an event is pinned and the banner displays the correct event.

Fix when an event is pinned and the banner displays the good event.

* Update e2e tests
…atrix-org#12946)

* Display the indicator even with one message

* Update e2e tests
* Fix read receipt animation

The way it was done involved remembering dom nodes and then getting
their position later when animating the receipt to its next position,
but I'm not sure how this worked since the DOM node may not neccessarily
be in the DOM anymore. Instead, just remember the bounding box coordinates.
At worst it might go weird if the window is resized but seems fine in
practice. Also, keeping references to dom nodes feels like a fast road
to memory leaks.

Fixes element-hq/element-web#27916

* Attempt to write a test for read receipts

and fix naming

* Another test

also change a condition to make it testable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.