toggldesktop icon indicating copy to clipboard operation
toggldesktop copied to clipboard

Adds a DatePicker to EditView.xaml for the End Date.

Open daiplusplus opened this issue 3 years ago • 7 comments

📒 Description

This change adds a second <DatePicker> to EditView.xaml which displays the Date component of an event's end timestamp and integrates it with EditView's plumbing.

For my motivation, see https://github.com/toggl-open-source/toggldesktop/issues/2582

🕶️ Types of changes

New feature (non-breaking change which adds functionality)

🤯 List of changes

  • The EditView.xaml now has this layout and UI: image

  • EditView.xaml.cs is updated with changes to integrate the End DatePicker.

  • Also fixes build issues with the Windows project by specifying updated paths for the Windows 10 SDK's *.winmd files. Other users can add their own paths as well, if necessary, without breaking the <HintPath> for other users.

👫 Relationships

https://github.com/toggl-open-source/toggldesktop/issues/2582

Closes #2582

🔎 Review hints

I did some cursory testing for both Existing and New entries. I ran all tests in the solution - I wanted to add some UI integration tests for this change but I didn't see any, unfortunately. I did have unit test failures that were unrelated to my changes.

I'm perplexed by the design of Toggl's native API - I don't know why it treats the end-time as a localized time-only string instead of passing around only UNIX timestamp integers, that would make the Toggl API much simpler to work with and easier to understand.

daiplusplus avatar Jan 23 '21 11:01 daiplusplus

@atsvirchkova @skel35 I noticed other PRs ahead of mine have already been reviewed and merged - so I was wondering where in your review pipeline (or kanban board) this PR is currently at. Thanks!

daiplusplus avatar Mar 07 '21 04:03 daiplusplus

@AndrewVebster @skel35 @atsvirchkova ping?

daiplusplus avatar Jun 16 '21 01:06 daiplusplus

@AndrewVebster @skel35 @atsvirchkova ping?

Is this project dead? I created a few issues, one of them was a year ago, all of them if no response, I came back here thinking about getting my hands dirty, but if the PRs are not being merged I lose a bit of interest.... Is the company abandoning this project and switching to a PWA? I think we should be aware of anything like this.... Edit. bunch of typos

rzfzr avatar Jul 20 '21 23:07 rzfzr

Hi @Jehoel @rzfzr! This project is in a partially frozen state. We're planning to have refreshed native apps in the future without our outdated C++ core which has piled a lot of tech debt by now. The new apps will also be closed-source at first, although this can change in the future. This is still our main desktop app at the moment and we can publish new releases if we have the need to. But since this project is not in active development, we generally don't want to publish any cosmetic or small updates to it and we treat any issues/requests as requests for the new apps.

skel35 avatar Jul 21 '21 09:07 skel35

@skel35 no worries, I cancelled my subscription to Toggl out of disgust for your company's contempt, or at least diregard, for open-source contributors ages ago :)

daiplusplus avatar Jul 21 '21 09:07 daiplusplus

@skel35 thanks for the quick reply! Would you be so kind to put this "crucial bug fixing" disclaimer first thing in the main Readme? Just feels fair, every non accepted PR is a competent someone that likes this product so much that they took time out of their day, and waste it... that's the recipe for getting reactions like Jehoel's...

rzfzr avatar Jul 21 '21 11:07 rzfzr

@rzfzr You're right that this information should've been shared publicly earlier. Thank you for the feedback. I've added a disclaimer to the top of the repo's Readme.

skel35 avatar Jul 21 '21 12:07 skel35