[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64023: 29.0.91; gnus-icalendar does not update timestamp when descri
From: |
Reindert-Jan Ekker |
Subject: |
bug#64023: 29.0.91; gnus-icalendar does not update timestamp when description is empty |
Date: |
Fri, 16 Jun 2023 20:45:48 +0200 |
User-agent: |
mu4e 1.10.3; emacs 29.0.90 |
As a little exercise I've been writing some unit tests for the bug you
fixed regarding updates to events with empty description. Patch is
attached. Looking forward to any feedback.
0001-test-lisp-gnus-gnus-icalendar-tests.el-Add-tests-for.patch
Description: Patch: unit tests
Robert Pluim <rpluim@gmail.com> writes:
> (please keep the bug address in CC)
>
>>>>>> On Mon, 12 Jun 2023 20:16:30 +0200, Reindert-Jan Ekker
>>>>>> <info@codesensei.nl> said:
>
> Reindert-Jan> Yes, that works!
>
> OK. Eli, emacs-29 or master? Itʼs a pretty small change, but itʼs not
> a regression as such.
>
> Reindert-Jan> By the way, what is the intended behaviour of
> Reindert-Jan> gnus-icalendar--cancel-org-event? It seems to search for a
> property "DT"
> Reindert-Jan> that does not exist (at least in my files), so it does not
> change the
> Reindert-Jan> event. Is this a bug or am I missing some configuration
> somewhere?
>
> It looks like a bug. I canʼt find any code creating such a property,
> the current code inserts the timestamp in the body of the org entry.
>
> Robert
--
Reindert-Jan Ekker
Code Sensei