Skip to content

XEP-0202 Entity Time#3978

Open
SeveFP wants to merge 7 commits intoconversejs:masterfrom
SeveFP:xep-0202-entity-time
Open

XEP-0202 Entity Time#3978
SeveFP wants to merge 7 commits intoconversejs:masterfrom
SeveFP:xep-0202-entity-time

Conversation

@SeveFP
Copy link
Copy Markdown
Member

@SeveFP SeveFP commented Mar 20, 2026

Used https://github.com/SeveFP/XEP-0202-test-bot/ to test the implementation.

image

No MUC support yet.

  • 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.

@SeveFP SeveFP requested a review from jcbrand March 20, 2026 15:39
@SeveFP SeveFP self-assigned this Mar 20, 2026
@bobpaw
Copy link
Copy Markdown
Contributor

bobpaw commented Mar 20, 2026

The DOAP should be updated with at least the status field (see XEP-0453 section 4.2).

@SeveFP
Copy link
Copy Markdown
Member Author

SeveFP commented Mar 23, 2026

image

@bobpaw
Copy link
Copy Markdown
Contributor

bobpaw commented Mar 23, 2026

My personal opinion is still that there should also be a send_entity_time option so that users can choose to share or not.

Also, I'm wondering if it makes sense to cache user's timezones when received (in browser storage or maybe private PEP)? Otherwise how useful is it for offline (or temporarily offline) users?

Comment thread src/headless/plugins/time/index.js Fixed
@jcbrand
Copy link
Copy Markdown
Member

jcbrand commented Mar 24, 2026

See Slack as an example of more user-friendly text and icon:

image

@SeveFP
Copy link
Copy Markdown
Member Author

SeveFP commented Mar 24, 2026

image

@jcbrand
Copy link
Copy Markdown
Member

jcbrand commented Mar 24, 2026

@SeveFP I find it funny that you're using it in Afrikaans 😃

(I know it's the default language in dev.html )

image

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.

4 participants