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

XMPP presence priority #38

Open
Neustradamus opened this issue Jan 6, 2019 · 6 comments
Open

XMPP presence priority #38

Neustradamus opened this issue Jan 6, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@Neustradamus
Copy link

Neustradamus commented Jan 6, 2019

Can you add the XMPP presence priority?
from -128 to +127.

@kevinfaveri kevinfaveri added the enhancement New feature or request label Feb 4, 2019
@kevinfaveri kevinfaveri self-assigned this Feb 4, 2019
@kevinfaveri
Copy link
Collaborator

kevinfaveri commented Feb 4, 2019

You mean when sending the presence stanzas? I could turn this into a parameter, like the URL parameter of the XMPP server.

@Neustradamus
Copy link
Author

@kevinfaveri
Copy link
Collaborator

From here: https://tools.ietf.org/html/rfc6121#section-4.7.2.3

Yeah it is sending the priority prop on presence stanzas... will look into a solution for this...

@kevinfaveri
Copy link
Collaborator

XMPP presence priority now defaults to +127

@Neustradamus
Copy link
Author

Why have you selected directly the max value by default?

It is possible to select manually in the webchat?
The user must select a value only between -128 and +127.

@kevinfaveri
Copy link
Collaborator

kevinfaveri commented Apr 27, 2019

I'm going to need to think about it better, but I do not think of leaving it configurable by the user as it will add more settings and the idea of Opa is not to support everything, but to be a simple web client for XMPP. If I can find a way that looks cool in the user experience I will add this functionality.

My concern is with the user experience, but as I intend to remake part of the UI in the future, I might add a configuration for priority...

So, I'm going to open this issue as I decide whether this request will be met in some way or whether the priority tag will be omitted (where 0 is assumed).

Opa is currently omitting the priority tag (so 0 should be assumed by XMPP specs).

@kevinfaveri kevinfaveri reopened this Apr 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants