Allow custom notification class for people who would like to use othe…

[maukoese]

I'd like to use SMS to send out the notifications, because some of my users don't have email addresses. I feel like it would help a lot for myself and other devs to be able to customize the notification class being used; to add their channels of choice, besides just email.

While in my case I'm doing SMS, any other channel including Whatsapp, Telegram of FCM can be used to send the OTP

[afsakar]

Thank you for your PR. 🤝 I will check asap.

[maukoese]

Looking forward to your feedback, and hopefully, merging.

[maukoese]

Hello. Any feedback on this?

[maukoese]

@afsakar just realized something in the migration:

//... other columns
$table->string('email'); <--- this one
//... other columns

What if we replaced the email column and/or added a user_id columns with would be easier to work with if the record doesn't have an email?

We could use the foreignIdFor() helper which works for strings(uuid, ulid) as well as integers.

So something like:

$table->foreignIdFor(\App\Models\User::class);

We can even go further and add the user model to config so people can use other models if they wish

Something like:

$table->foreignIdFor(config('filament-otp-login.user_model', \App\Models\User::class));

[afsakar]

I don't have any plans at the moment, but I'm thinking of implementing these after v4 is released. But thank you anyway 🙏

[maukoese]

Alright, thank you so much.

I'm avaiable if you need any help with the implmentation and support.