Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kimai v2 latest #22

Conversation

vitormattos
Copy link

Hi @KatjaGlassConsulting
I fixed more issues and I think that #21 was solved now (but I didn't tested).

Now, I identified a problem when reject a week.
When I click at button "reject", don't do nothing.

But, I was able to approve a work week now with success.

I followed doing all commits separated to be more easy to do the review.

Try to solve this problem at log:

> User Deprecated: Context: Passing values to option `--all-or-nothing` Problem: Passing values is deprecated

Identified that this prolem stay at Kimai side, but is good to upgrade.

ref: doctrine/migrations#1383

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
When we access a key of array that don't exists, throw an exception, to
prevent this I used the empty constructor

Signed-off-by: Vitor Mattos <[email protected]>
the most fast way to do this but not the best way because isn't locally

Signed-off-by: Vitor Mattos <[email protected]>
Use server timezone when the use haven't a timezone

Signed-off-by: Vitor Mattos <[email protected]>
@KatjaGlassConsulting
Copy link
Owner

The overtime also has some issues. I am still struggeling with the API. Does these work in your environment? You can to click your profile -> API access and then click the book icon "swagger api doc". When you have defined an api key, on the swagger api you can test it by authorization (use your username and api passoword) and then go to the api endpoint to test.

The swagger API doc does not open for me "{"code":500,"message":"Internal Server Error"}" - when I remove all approval APIs (rename .php to .php_ for example), then the doc is up and running.

Nevertheless, I already merge so that can be used as base.

@KatjaGlassConsulting KatjaGlassConsulting merged commit 22ab6cc into KatjaGlassConsulting:kimai_v2_update Dec 6, 2023
0 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants