Link Clicked Events not being "caught" by Listener about short-url

[ash-jc-allen]

Hey! Great spot, I'm glad you managed to fix that.

If you're still running into any issues with trying to access the visit property, feel free to give me a shout and I'll see if I can lend a hand at all 😄

[ash-jc-allen]

Hey! Sorry that you're having problems with the package. I'll take a look into this 😄

I did notice in your EventServiceProvider example that the PhoneMessageLinkClicked listener looks like is being imported with a namespace that looks like a Laravel vendor namespace (use Illuminate\Auth\Listeners\PhoneMessageLinkClicked;). Is that the actual namespace of the listener you're trying to import?

I'm just wondering if there's an error in the example you've sent over, or if that's intentional 🙂

[HeadStudios]

Hi Ash,

Thank you for your quick response - and you're absolutely right my namespacing convention for the my Listener was incorrect in the EventServiceProvider file - I have just updated it as per your instructions which you can see in the Gist below:
https://gist.github.com/HeadStudios/777e2375d99a68cb2cf0f43550ace1a5

I then created another shortlink and visited it however can confirm the event is still not firing - I really appreciate your input and any other ideas you have would be much appreciated.

[HeadStudios]

Hi Ash,

Just updating that I have fixed the event firing action - it was due to the namespace of the Event itself which I have updated to:
use AshAllenDesign\ShortURL\Events\ShortURLVisited;

The only challenge now is the handle() functionality throwing an error for this code:
https://gist.github.com/HeadStudios/7e4a2d499ef31d45542d003b862ff25c

With the error:
Undefined property: AshAllenDesign\ShortURL\Events\ShortURLVisited::$visit

However I am confident I will be able to take it from here - thank you for pointing me to the use statements being an issue and I appreciate your quick response and help!

I will mark this as Closed.