Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Remove Riot beta warning badge #1004

Closed
wants to merge 1 commit into from
Closed

Remove Riot beta warning badge #1004

wants to merge 1 commit into from

Conversation

Perelandra0x309
Copy link
Contributor

Description

Resolves: #none

Riot is no longer considered in beta:
https://matrix.org/blog/2019/06/11/introducing-matrix-1-0-and-the-matrix-org-foundation

We are very excited to announce the first fully stable release of the Matrix protocol and specification across all APIs - as well as the Synapse 1.0 reference implementation which implements the full Matrix 1.0 API surface.

This means that after just over 5 years since the initial work on Matrix began, we are proud to have finally exited beta!! This is the conclusion of the work which we announced at FOSDEM 2019 when we cut the first stable release of the Server-Server API and began the Synapse 0.99 release series in anticipation of releasing a 1.0.

@Perelandra0x309 Perelandra0x309 requested a review from a team June 22, 2019 12:19
@netlify
Copy link

netlify bot commented Jun 22, 2019

Deploy preview for privacytools-io ready!

Built with commit c925e97

https://deploy-preview-1004--privacytools-io.netlify.com

@ghost
Copy link

ghost commented Jun 22, 2019

While Riot.im is the reference client implementation it is neither the Matrix specification or the reference server implementation (Synapse).

@ghost
Copy link

ghost commented Jun 22, 2019

While Riot.im is the reference client implementation it is neither the Matrix specification or the reference server implementation (Synapse).

Maybe I should have read things a bit more closely. From the quoted part my previous comment would have been correct but looking at the link there's a link to https://medium.com/@RiotChat/the-big-1-0-68fa7c6050be saying Riot.im is out of beta (back in February) so I guess the badge should be removed after all.

@Perelandra0x309
Copy link
Contributor Author

Yes it would have been more correct for me to write Riot/Matrix is out of beta, considering the entire system. Riot itself has been out of beta for a while but Matrix not until recently.

Copy link
Contributor

@Mikaela Mikaela left a comment

Choose a reason for hiding this comment

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

I don't think this can be merged until element-hq/element-web#6779 is resolved as the warning refers to it.

@ghost
Copy link

ghost commented Jun 22, 2019

I don't think this can be merged until vector-im/riot-web#6779 is resolved as the warning refers to it.

Wouldn't that more correctly be needing an experimental badge instead?

@Mikaela
Copy link
Contributor

Mikaela commented Jun 22, 2019

Sure, I would be fine with changing it to experimental even if it would hopefully soon be removed if the Matrix/Riot people aren't being too optimistic.

@erciccione
Copy link

I would still leave the warning. As @Mikaela pointed out, many privacy related concerns are still unresolved.

@Perelandra0x309
Copy link
Contributor Author

Here's a good update on what the developer considers outstanding issues.
https://github.com/privacytoolsIO/privacytools.io/pull/562#issuecomment-502681069

Perhaps it is best to change the badge text and tooltip to something besides beta, which at this point is not correct. It could be a "Warning" and a tooltip about E2EE having these issues from the above comment:

E2E search (which is almost done, as of last week)

This seems like just an ease of use issue, not a privacy concern.

Cross-signing to simplify verification (demoable as of last week)

Another ease of use issue which will be nice to have.

Support for non-E2E clients via a shim (working as of a few weeks ago).

This is a new feature which allows some intermediate server to act as the E2E endpoint that a non E2E enable client connects to.

Metadata is stored on servers. Fixing this is a long term project.

This is worth mentioning as a warning.

We need to better at helping users understand where their contact data goes, and hash contact data during lookups

Another thing worth mentioning.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants