Unable to send notifications

[ericp1337]

I am getting an error saying that the fields from_id,to_id,category_id are required using this code.
This used to work before, so I am not sure why it doesn't anymore after the new version. Has something changed?

Notifynder::loop($friends, function(NotifynderBuilder $theBuilder, $user) {
                $theBuilder
                    ->category('user.situation.updated')
                    ->to($user)
                    ->from($this->user)
                    ->url(url()->current())
                    ->extra(['username' => $this->user->user_name])
                    ->expire(Carbon::now()->addDay());
            })->sendWithPushNotification();

[Gummibeer]

Hey,
first yes: there are some config updates but they shouldn't conflict with the sending.

First Tip: go back to Notifynder 3.1.6 this is the latest version before these updates.

Now to the solution, I will try to reproduce it and fix it after this. More details in slack.

[Gummibeer]

Ok, cause you have an empty collection in the loop it throws an Exception in the send() method.

In PR #119 I've updated it to throw an InvalidArgumentException in the loop() method itself if it's empty.
Can you pls test it if this solves your problem (throws the right Exception) - composer version is dev-issue-118

[Gummibeer]

ToDo:

• split cases
• precise Exception for not iterable
• precise Exception for empty

[Gummibeer]

PR #119 is merged - issue done & closed!?