Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Move the cross-signing reset to its own page #2364

Merged
merged 4 commits into from
Feb 17, 2024

Conversation

sandhose
Copy link
Member

@sandhose sandhose commented Feb 16, 2024

This fixes a bunch of small issues, see each commit:

  • Upgrade compound-web to remove React warning
  • Make sure the error boundary & loading spinner don't suspend
  • Make sure all the loader use the abort signal
  • Move the cross signing reset UI in its own page

Also adds a deep-link to the page, via ?action=org.matrix.cross_signing_reset

Screenshot 2024-02-16 at 19 42 00 Screenshot 2024-02-16 at 19 42 31

The Back button goes back home, and if the user got to this page via deep-linking, the back button gets awat

@sandhose sandhose added the A-Frontend Changes on the React frontend label Feb 16, 2024
@sandhose sandhose requested a review from t3chguy February 16, 2024 18:28
Copy link

cloudflare-workers-and-pages bot commented Feb 16, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6431d14
Status: ✅  Deploy successful!
Preview URL: https://d85680bb.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-frontend-cross-sign.matrix-authentication-service-docs.pages.dev

View logs

This is done by making it possible to render the GenericError, Layout
and Footer with a `dontSuspend` prop. This then sets the `useSuspense`
to `false` in translations, and sets a default translations in case they
haven't been loaded yet.
This makes sure that the loader stops loading when navigating away
@sandhose sandhose force-pushed the quenting/frontend/cross-signing branch 2 times, most recently from 30327cb to 4fa7888 Compare February 16, 2024 18:34
@sandhose sandhose force-pushed the quenting/frontend/cross-signing branch from 4fa7888 to 6431d14 Compare February 17, 2024 08:42
@sandhose sandhose enabled auto-merge (rebase) February 17, 2024 08:45
@sandhose sandhose merged commit aefcc3c into main Feb 17, 2024
16 of 18 checks passed
@sandhose sandhose deleted the quenting/frontend/cross-signing branch February 28, 2024 16:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Frontend Changes on the React frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants