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

Add fluffychat://#room and fluffychat://@user URIs #47

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@ChristianPauly
Copy link

commented Oct 27, 2018

On a Ubuntu Touch device with fluffychat installed, you can click on these URIs and it will open fluffychat automatically.

@KitsuneRal

This comment has been minimized.

Copy link
Member

commented Oct 27, 2018

Can we please do matrix-org/matrix-doc#455 instead?

@aaronraimist

This comment has been minimized.

Copy link
Contributor

commented Dec 13, 2018

@KitsuneRal yes that is the dream but it isn't here yet. the matrix.to page itself says

"Links are designed to be human-friendly, both for reading and constructing, and are essentially a compatibility step in the journey towards a ubiquitous mx:// URL scheme."

I'd say merge this for now and then when mx:// is here you won't really need matrix.to anymore.

@KitsuneRal

This comment has been minimized.

Copy link
Member

commented Dec 14, 2018

@aaronraimist, it's not a dream; it's a design that, it seems, only me is really pushing forward. If this goes in, I'll have to quickly draft quaternion:// (and spectral://, serving interests of QMatrixClient ecosystem) in a similar vein.
So can we please mature matrix-org/matrix-doc#455 instead and then add matrix:// to matrix.to? "Not needing" matrix.to is not going to happen anytime soon, now that we have plenty of links with it already in room DAGs.

@aaronraimist

This comment has been minimized.

Copy link
Contributor

commented Dec 14, 2018

It's a dream in the sense that everyone wants it but it doesn't exist. I'm with you that yes it should exist.

I'm not anyone special so all I can do (and did do months ago) is upvote that issue.

What can be done to push that proposal forward? Don't you need to write it up as an MSC? (I'm not familiar enough with the process, yes there is an issue but don't you need to PR?)

@KitsuneRal

This comment has been minimized.

Copy link
Member

commented Dec 14, 2018

It already is an MSC. As of now, I'm looking for feedback on its contents - whether something is missing that should be added, whether it's (not) consistent enough, etc.

I was pushed sidetracks by having to urgently work on lazy-loading in QMatrixClient; but as of yesterday, I'm back from that - which means that, unless I get some track-changing feedback along the way, I will add matrix:// to libQMatrixClient and Quaternion in the nearest couple of weeks - just enough to check there are no elephants in the room. Once this is done, I'll ask for final comments period to start, which takes another week, if there are no blocking concerns. After FCP the proposal will be merged and the PR for the spec will have to submitted. Roughly at the same time, I expect to also PR matrix:// support in matrix.to - which means sometime January.

@benparsons

This comment has been minimized.

Copy link
Member

commented Jul 1, 2019

We should not have delayed on merging this. Are there any reasons not to accept a request like this, even knowing that matrix-org/matrix-doc#455 remains the true end-goal?

@KitsuneRal

This comment has been minimized.

Copy link
Member

commented Jul 1, 2019

Why not add matrix://, again?

@benparsons

This comment has been minimized.

Copy link
Member

commented Jul 1, 2019

The intention is always to add matrix://, but right now, that would mean waiting on MSC455. Would it be appropriate to introduce matrix:// before then? If not, then there seems to be no reason to block use of fluffychat:// while waiting.

@aaronraimist

This comment has been minimized.

Copy link
Contributor

commented Jul 1, 2019

@KitsuneRal is there any status update on matrix://? It would be nice to push that along. Are you waiting on others to try implementing it?

@KitsuneRal

This comment has been minimized.

Copy link
Member

commented Jul 1, 2019

I agree the entire thing is well overdue, so here goes.

  1. I don't see why we can't add matrix: (or even mx:) scheme to matrix.to at this stage. The scheme defined in the MSC is now down to a single option, and this option hasn't really evolved for the last few months. It needs not be final to be added to matrix.to; moreover, I think it's time to give a bit of real-world testing by adding it to matrix.to.
  2. To that end, I have just enabled Quaternion to open matrix: URIs in-application: quotient-im/Quaternion@a711e60

Let's push that envelope already.

@ChristianPauly

This comment has been minimized.

Copy link
Author

commented Jul 2, 2019

fluffychat:// is just a workaround for a bug in Ubuntu Touch which prevents https links in the url-dispatcher. I'm not sure if this is my fault or its a bug in the system itself (documentation is not that clear for me). However in the future I would also like to replace it with a more universal solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.