VacationDays icon indicating copy to clipboard operation
VacationDays copied to clipboard

Single days, half days and how much is left

Open chris-asdf opened this issue 2 years ago • 17 comments

Hi,

this app seems to be a nice addition to my current way to track my work time. But, three questions remain:

  1. How can I conveniently add one single holiday /day off? If the start and end date are the same, there is a zero in front of the entry
  2. Is it possible to add half days?
  3. How can I enter the number of holidays I have for a year and how much is left?

Thanks in advance, all the best Chris

chris-asdf avatar Aug 13 '22 18:08 chris-asdf

Hey, thanks for your feedback. 😊

  1. I don't see this. When I add a holiday for August 15 I see a 15 in front of the entry. May you please share a screenshot?

  2. No, that is not possible at the moment. What do you think of adding this feature, @arafaatqureshi & @amarradi?

  3. This feature is planned for one of the next releases.

CrazyMarvin avatar Aug 14 '22 00:08 CrazyMarvin

will be updating the total holidays count in the upcoming release. Thanks

arafaatqureshi avatar Aug 14 '22 06:08 arafaatqureshi

Hi,

that's great news, that you are planning to add these features!

Regarding the single day I've attached two screenshots when I add a vacation for 08/01 . On the date picker screen I'm not allowed to save it until I set an end date, which I set also to 08/01.

All the best Chris

Screenshot_20220814-143255_Vacation Days Screenshot_20220814-143301_Vacation Days

chris-asdf avatar Aug 14 '22 19:08 chris-asdf

Hello @chris-asdf, I have fixed the date display for vacations with a single day. https://github.com/unbiaseduser/VacationDays/commit/7da5b7ab49ce7175dcff3d8efc8ac38dca43a7dd Can you try adding such a holiday with this APK? In your case, the date displayed should be "01" instead of "0". You have to uninstall the original app before installing this one! app-debug.zip

unbiaseduser avatar Sep 17 '22 02:09 unbiaseduser

@chris-asdf we will be looking for some other way too, but for now as you have to enter the end date as well. so, the same date would be the end date for the single holiday. and then save it, it will let you save .

arafaatqureshi avatar Sep 17 '22 07:09 arafaatqureshi

@arafaatqureshi Are you going to check my code?

unbiaseduser avatar Sep 17 '22 07:09 unbiaseduser

@unbiaseduser sure.

arafaatqureshi avatar Sep 17 '22 09:09 arafaatqureshi

Hi,

thanks @unbiaseduser for fixing. I just give the apk you attached a try and this will do it for me for the single days. I could also imagine a way of entering no end date if a corresponding checkbox is clicked, that defines if it's a single holiday. But that's not that important.

Also thanks - whoever did this 😃 - for adding the feature with the configurable number of holidays.

Any chance for half days?

All the best Chris

chris-asdf avatar Sep 20 '22 20:09 chris-asdf

@chris-asdf you're welcome. Also I would like to mention that the broken date display issue and the feature to conveniently add 1 holiday issue are completely different issues. Which means that if I were to add such feature without fixing the date display, it would still be broken. Speaking of that feature, I might add it somewhere down the line. (As for the half days one, I haven't got any ideas how to implement it yet.)

unbiaseduser avatar Sep 21 '22 00:09 unbiaseduser

Hey, what about adding half-days ? Could I help on making this feature ? I'm not sure, I can do it soon tho, but I can always keep struggling (which is always great with open source stuff)

AkechiShiro avatar Oct 25 '22 06:10 AkechiShiro

Hey, what about adding half-days ? Could I help on making this feature ? I'm not sure, I can do it soon tho, but I can always keep struggling (which is always great with open source stuff)

Sure, go for it, no need to ask

unbiaseduser avatar Oct 25 '22 07:10 unbiaseduser

Thanks for your offer to help, @AkechiShiro! That's awesome!

CrazyMarvin avatar Oct 25 '22 07:10 CrazyMarvin

Are you still up for it, @AkechiShiro? Next month is Hacktoberfest by the way. 😅

CrazyMarvin avatar Sep 26 '23 22:09 CrazyMarvin

Hi @CrazyMarvin

Very sorry for not having gave any update on this, I completely forgot about this, and got busy in life on a few others projects, at the moment I'm short on storage space, but would need Android Studio to properly develop this feature, I believe.

I will definitely try and take a look at what needs to be modified, but this will take me time as I'm not familiar with the code base at all.

AkechiShiro avatar Sep 26 '23 23:09 AkechiShiro

Are you interested to work on the feature to support half days, @karimapps?

CrazyMarvin avatar Jan 24 '24 21:01 CrazyMarvin

I'm sorry I couldn't get to take a full stab at this issue @CrazyMarvin

AkechiShiro avatar Jan 24 '24 23:01 AkechiShiro

I'm sorry I couldn't get to take a full stab at this issue @CrazyMarvin

Don't worry.

CrazyMarvin avatar Jan 25 '24 19:01 CrazyMarvin