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

Updates #2586: Add support for XEP-0402 Bookmarks #3562

Merged
merged 5 commits into from
Dec 21, 2024
Merged

Conversation

jcbrand
Copy link
Member

@jcbrand jcbrand commented Dec 8, 2024

No description provided.

@jcbrand jcbrand force-pushed the jcbrand/xep-402 branch 4 times, most recently from 8c8648d to 1936790 Compare December 13, 2024 06:03
@jcbrand jcbrand force-pushed the jcbrand/xep-402 branch 2 times, most recently from 3164eb2 to 975eca8 Compare December 14, 2024 07:39

const { Strophe, $iq, sizzle } = converse.env;

const { Strophe, sizzle, stx } = converse.env;

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused variable sizzle.
@jcbrand jcbrand force-pushed the jcbrand/xep-402 branch 2 times, most recently from 3cd1028 to a825314 Compare December 18, 2024 06:54
To bring it in line with other chat boxes.
- Update to latest Strophe.js
- Add support for setting the password in the bookmark
- Add more explicit parsing of bookmarks from a stanza
- Save and include the `<extensions>` element in outgoing bookmark stanzas
- Turn it into a folder
- Use the stx tagged template literal
@jcbrand jcbrand changed the title WIP: Updates #2586: Add support for XEP-0402 Bookmarks Updates #2586: Add support for XEP-0402 Bookmarks Dec 21, 2024
@jcbrand jcbrand merged commit acc5427 into master Dec 21, 2024
4 checks passed
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.

1 participant