logseq / marketplace

A centralized packages manager for Logseq marketplace plugins.

MIT License

253 stars 282 forks source link

add logseq-task-done-time-mini plugin package #560

Closed mlhiter closed 3 months ago

mlhiter commented 3 months ago

Submit a new Plugin to Marketplace

Plugin Github repo URL: https://github.com/mlhiter/logseq-task-done-time-mini

Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (theme plugin for this).
[x] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

xyhp915 commented 3 months ago

Hi @mlhiter Thank you for sharing such a useful plug-in :) Just testing that was found the event listened to by db change entered into an infinite loop.

https://github.com/user-attachments/assets/aa84cee6-8660-4528-a85b-a9c7bdab75be

mlhiter commented 3 months ago

This seems my regexp error，I will fix this error this afternoon.💪

mlhiter commented 3 months ago

@xyhp915 I fixed this bug just now.😀

xyhp915 commented 3 months ago

@xyhp915 I fixed this bug just now.😀

I still have this issue; you can try using the preferredDate configuration format as below to reproduce! CleanShot 2024-08-02 at 15 59 39@2x

BTW, I probably guess it's caused by these two related regular expressions.

CleanShot 2024-08-02 at 16 05 07@2x

Helpful links: https://github.com/logseq/logseq/blob/7cc0b15d0436d025527cbd9108a5951fef54552f/src/main/frontend/worker/date.cljs#L14

mlhiter commented 3 months ago

@xyhp915 Yes,you are right. A comma is always followed by a space and I forget to modify the value.hh. And now its no problem : ) v1.1.2 is released.