Hugh Nimmo-Smith

Results 30 comments of Hugh Nimmo-Smith

FTR, the latest iteration on the design shows the hostname of the server hosting the management URL:

> @hughns err, I'm not clear on why my review is requested? Because you previously reviewed this PR and commented. The PR is old, so feel free to take yourself...

@americanrefugee this is what I have implemented for next iteration: ![image](https://user-images.githubusercontent.com/6955675/214100825-b78824c6-cf54-45a2-874d-486bbf4de048.png) Notes: - I've tweaked the wording to reference "managed" have than "stored" - The horizontal line isn't present anywhere...

I've added references to the following MSCs that this proposal assumes are generally accepted: - https://github.com/matrix-org/matrix-spec-proposals/pull/3967 - https://github.com/matrix-org/matrix-spec-proposals/pull/3970

The Web support has been merged so I think that this is ready for review. Android implementation is done too but not yet merged.

Closing this in preference to the following smaller PRs: - https://github.com/vector-im/hydrogen-web/pull/983 - https://github.com/vector-im/hydrogen-web/pull/948 - https://github.com/vector-im/hydrogen-web/pull/982 - https://github.com/vector-im/hydrogen-web/pull/980 I've parked running nginx as non-root for now.

It is proposed that [MSC4108](https://github.com/matrix-org/matrix-spec-proposals/pull/4108) supersedes this MSC.

Closing this PR as https://github.com/matrix-org/matrix-spec-proposals/pull/4108 is now ready for review.

@ldb2000 if you are using brew to install graphviz then it looks like you need to install with: `brew install graphviz --with-pango` See https://stackoverflow.com/questions/48383874/graphviz-crashes-on-mac-during-render-using-quartz and https://github.com/Homebrew/homebrew-core/issues/18684

> I'd suggest adding a new api `encryptToDeviceMessages` to `CryptoApi`, which takes an array of `(user_id, device_id)` pairs. The application can then call `MatrixClient.queueToDevice`on the results. This sounds perfect 👍