[BUG] invoices are not shown - timesheet entries are not shown in calender about kimai2

[tobybatch]

I will take a look but this may be an issue for kimai app itself,

[stevenhrabok]

Hi @tobybatch, I'm experiencing the same issue (when preview or print invoice).

I attempted to look at the kimai and nginx container logs and it seems we only see HTTP 200's.
When I click Preview I get http://mykimaihost:8001/en/invoice/?searchTerm=&daterange=2020-01-01+-[phone]&customer=1&project=1&activity=1&users%5B%5D=1&tags=&exported=1&template=1&preview=
When I do print I get http://mykimaihost:8001/en/invoice/?searchTerm=&daterange=2020-01-01+-[phone]&customer=1&project=1&activity=1&users%5B%5D=1&tags=&exported=1&template=1&create=

When searching Kimai Github Issues, I'm not seeing any bug reports regarding this issue.
Is there anything I can provide to assist to find the source of the problem?

[kevinpapst]

@stevenhrabok can you please test some POST forms (edit/create something). Do they work?
Now test some forms that utilize GET parameter (eg. search, invoice, pagination). Do they work?
I'd say #66 is related.

[tobybatch]

@Schrolli91 have you time to look at this? It could be nginx an forwarding issue, and I'm slammed at work right now.

[stevenhrabok]

@tobybatch Found the fix, attempted to submit a branch with PR, but didn't have permission.

https://github.com/tobybatch/kimai2/blob/master/compose/nginx_site.conf#L14

needs to be changed to try_files $uri $uri/ /index.php$is_args$args;

the missing $is_args$args was causing the uri arguments to be lost.

[kevinpapst]

@stevenhrabok awesome, nice finding! I looked through it, but didn't spot it.
FYI - you have to fork the repository, then create the branch, do your changes, push the branch. Now you can create a PR

[tobybatch]

Top work, thanks you. I'll get that fix in place over the weekend

[Schrolli91]

ok - I'm too late :-D
@stevenhrabok thank you for your investigations

[AlxBe]

Perfect! Thank you very much.

[tobybatch]

@Schrolli91 Can you fix this please? I'm a nginx n00b and I can seem to get it to work.

[Schrolli91]

@stevenhrabok can you please take a look at the PR #71

atm i'm not able to test these changes properly - should be confirmed before we'll merge this to the master branch

[Schrolli91]

closed because PR is on the way