Patrick Brouwers comments

Results 142 comments of


                                            Patrick Brouwers

trafficstars

Missing documentation

Indeed never got the change to finish the docs. The example in the docs does have the example of adding groups and items: ``` $this->menu->group('settings', function(Group $group) { $group->item('User Settings');...

Update drawings.md

Can you update it to https://phpspreadsheet.readthedocs.io/en/latest/topics/recipes/#add-a-drawing-to-a-worksheet

Missed attribute 'prices_are_incl_tax' for Estimate entity.

I created a PR for it: https://github.com/picqer/moneybird-php-client/pull/253

[Bug]: touch(): Unable to create file /app/storage/tmp/laravel-excel/laravel-excel-Lbgb6nfeSe1Hp9kL7ucC2qfZC3jFTNGX.xlsx because No such file or directory

Feel free to PR an improvement.

[Bug]: Charts on Multiple sheets not working when queued

Please try to provide a reproduction repository so I can easily test what is going wrong.

[Bug]: doesnt work with Laravel 8?

Cannot reproduce as bug. If you need help, please use the discussions tab.

[PROPOSAL] Remove memory caching of entity listeners

Feel free to PR it

[BUG] Unit tests not working when using :memory: driver sqlite database.

Cannot reproduce

[BUG] Unit tests not working when using :memory: driver sqlite database.

You cannot use that trait as indeed we don't share the PDOConnection

[1.*] Add mongodb support

If someone uses MongoDB and can helps us, we could start a `laravel-doctrine/odm` package. I never use it, so can't say it's high on my todo-list to start working on...