Skip to content

Conversation

@marcellintacite
Copy link
Contributor

Here's a draft for this issue : #1247
This merge request adds support for handling xmpp: protocol links in Converse.js, allowing users to click links like xmpp:user@example.com to automatically open a chat with the specified JID if logged in. This enhances user experience by enabling seamless integration with external XMPP links (e.g., from emails, websites, or other apps).

Before submitting your request, please make sure the following conditions are met:

  • Add a changelog entry for your change in CHANGES.md
  • When adding a configuration variable, please make sure to
    document it in docs/source/configuration.rst
  • Please add a test for your change. Tests can be run in the commandline
    with make check or you can run them in the browser by running make serve
    and then opening http://localhost:8000/tests.html.

@marcellintacite marcellintacite marked this pull request as draft October 1, 2025 08:05
@jcbrand
Copy link
Member

jcbrand commented Oct 7, 2025

Looking good so far @marcellintacite!

Please also remember to update the DOAP file to mention support for the new XEP
https://github.com/conversejs/converse.js/blob/master/conversejs.doap

marcellintacite and others added 4 commits October 7, 2025 16:01
Co-authored-by: JC Brand <jc@opkode.com>
Co-authored-by: JC Brand <jc@opkode.com>
Co-authored-by: JC Brand <jc@opkode.com>
@marcellintacite
Copy link
Contributor Author

@jcbrand Hello, i would like your review for the test i wrote

@marcellintacite
Copy link
Contributor Author

@jcbrand i need your help on tests so i can get done with this PR. I am facing some errors

@marcellintacite marcellintacite marked this pull request as ready for review November 10, 2025 10:47
@marcellintacite marcellintacite force-pushed the add-protocol-handler branch 2 times, most recently from 67fa8c7 to cd88326 Compare November 10, 2025 10:55
@marcellintacite
Copy link
Contributor Author

Hello @jcbrand i have fixed test issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants