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

Fix: timestamp in org-web-tools-read-url-as-org #54

Closed
wants to merge 1 commit into from
Closed

Fix: timestamp in org-web-tools-read-url-as-org #54

wants to merge 1 commit into from

Conversation

p4v4n
Copy link

@p4v4n p4v4n commented Mar 22, 2023

  • based on the new value of org-time-stamp-formats

The value of org-time-stamp-formats is updated on the latest master.
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/tree/lisp/org.el#n474

  - based on the new value of org-time-stamp-formats
@alphapapa
Copy link
Owner

What effect will this change have on users still using versions of Org in the wild?

@p4v4n
Copy link
Author

p4v4n commented Mar 22, 2023

What effect will this change have on users still using versions of Org in the wild?

Users with the old versions will additionally have arrow brackets(<>) around the timestamp after this change.

Edit: If you want to support the same behaviour for older versions perhaps we can directly use "%Y-%m-%d %a %H:%M" (or define a new variable) instead of using org-time-stamp-formats as it's only used currently at 1 place.

@ra1nb0w
Copy link

ra1nb0w commented Oct 30, 2023

Any update on this? thanks

@alphapapa alphapapa closed this in 69d161b Dec 20, 2023
@alphapapa
Copy link
Owner

Seems like org-time-stamp-format is the best function to use here. It's been in Org for a long time.

@p4v4n: Thanks for reporting this.

@ra1nb0w: Should work now. Let me know if not.

@p4v4n p4v4n deleted the fix-timestamp branch December 21, 2023 00:32
@ra1nb0w
Copy link

ra1nb0w commented Dec 21, 2023

It is working fine. Thank you very much!

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

Successfully merging this pull request may close these issues.

3 participants