Julio Motol

> This will break any existing installation that only use the built-in models and don't extend it. I don't see a reason to remove the folder attribute in the base...

The `$folder` properties on the built-in models has been reverted. Although I'd like to reiterate the potential breaking change I mentioned before: > ### Potential a breaking change > >...

I didn't thought anyone would do this so i didn't included it. If I may add, place the generated command in this directory. laravel-project/ app/ └── Modules/ └── FooBar ├──...

This package doesn't include any JavaScript logic to toggle between schemes. It uses CSS to detect the device's natively preferred color scheme. https://github.com/jloh/bulma-prefers-dark/blob/940dbb2d8891b2094c858781bef4c5f140664b45/sass/utilities/mixins.sass#L3-L5 If you want to toggle between the...

Hi, Sharing some of my findings ### First Controller Action SweetAlert::error('Some error'); dd(Session::get('sweet_alert')); // Displays the sweet alert config return redirect()->route('some-route'); ### 'some-route' Controller action dd(Session::get('sweet_alert')); // Displays 'null' The...

Hi, I found the problem with the facades. the `__destruct` method is not triggered. https://github.com/uxweb/sweet-alert/blob/fb88b736c86131c18254216e79559f4d93e08ca6/src/SweetAlert/SweetAlertNotifier.php#L360-L363 I then tried placing the `$this->flashConfig()` inside `message()` method and it works as intended.

> did you put "$this->flashConfig();" in the __destruct() or did you remove it? Neither, I was just trying to figure out what could have been the problem and tried hacking...

Specific exceptions for failed request would be nice! Could look something like below: try{ $google_places_response = GooglePlaces::nearbySearch($location, $radius, $params); }catch(OverQueryLimitException $e){ // handle OVER_QUERY_LIMIT response error }catch(RequestDeniedException $e){ // handle...

Before, when we used `league/fractal` (through `spatie/laravel-fractal`), they have a feature where you could set the [Default Includes](https://fractal.thephpleague.com/transformers/#default-includes) for a resource. This plays well with the spec as it also...

2 years after, no one is still merged the PR.