~psic4t/qcal#7: 
Display Day of Week next to date

When using qcal I was missing the option to display the day of week next to the date, since I tend to not know what the current date is.

Anyways, I implemented a very minimal solution that works for me here: https://github.com/corite/qcal . I did not (yet) hide it behind an option, although it might make sense to do so before adding this to mainline. Additionally to the above mentioned feature I updated Go and migrated some deprecated library calls.

This ticket is mainly intended to inquire whether you want to any of the changes mentioned above. If so, I'd be happy to work in some additional requirements (if you have any).

Status
RESOLVED CLOSED
Submitter
~corite
Assigned to
No-one
Submitted
7 months ago
Updated
6 months ago
Labels
No labels applied.

Matthias Debus 7 months ago ยท edit

Sound good, but I would spend an option for this (what about -wd ?). If you provide a patch with this, I'll merge it.

As for the library calls and other fixes(?) can you please provide single patches as well?

#Best regards psic4t

contact: xmpp:psic4t@data.haus nostr:psic4t@data.haus

Am 30.06.24 16:34, schrieb ~corite:

When using qcal I was missing the option to display the day of week next to the date, since I tend to not know what the current date is.

Anyways, I implemented a very minimal solution that works for me here: https://github.com/corite/qcal . I did not (yet) hide it behind an option, although it might make sense to do so before adding this to mainline. Additionally to the above mentioned feature I updated Go and migrated some deprecated library calls.

This ticket is mainly intended to inquire whether you want to any of the changes mentioned above. If so, I'd be happy to work in some additional requirements (if you have any).

-- View on the web: https://todo.sr.ht/~psic4t/qcal/7

Matthias Debus referenced this from #7 7 months ago

~corite 7 months ago

Ok cool. I have implemented the changes on this branch in my repo: https://github.com/corite/qcal/tree/feature/weeday-and-updates

The Go update and the feature are in two separate commits. You will also find the associated patches as attachments to this email if you prefer this format.

Best Regards,

corite

On 03.07.24 11:08, ~Matthias Debus wrote:

Sound good, but I would spend an option for this (what about -wd ?). If you provide a patch with this, I'll merge it.

As for the library calls and other fixes(?) can you please provide single patches as well?

Best regards psic4t

~corite 6 months ago

Hi, just a friendly inquiry whether you had the chance to look at this, and if there is anything left to do before merging :). (Also If you could create a new release after everything is merged that would be great!)

~psic4t 6 months ago*

Hey! Sorry for the late response! I was on a longer holiday.

There were no patches attached. Can you send them again (or upload on a pastebin)? This project is not on Github, so I can't merge your changes directly.

Best Regards, psic4t

~psic4t 6 months ago

Hey again,

just found out how to get diffs off commits on Github without cloning. :)

Your patches are merged in new version 0.9.2. Thanks!

~psic4t REPORTED CLOSED 6 months ago

Register here or Log in to comment, or comment via email.