Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add phone to message object #41

Closed
kepka42 opened this issue Mar 11, 2020 · 4 comments
Closed

Add phone to message object #41

kepka42 opened this issue Mar 11, 2020 · 4 comments

Comments

@kepka42
Copy link

kepka42 commented Mar 11, 2020

It happens that you need to directly specify the recipient in the message object. (For example, a similar functionality is in the library for working with Firebase Cloud Messaging). This is necessary when we work with AnonymousNotifiable.

@jhaoda
Copy link
Member

jhaoda commented Mar 11, 2020

(new AnonymousNotifiable())
    ->route('smscru', $order->getCustomerPhone())
    ->notify(new OrderArrived($order));

Чем не устраивает?

@kepka42
Copy link
Author

kepka42 commented Mar 12, 2020

Вариант вполне устраивает, но не в нашем случае. Мы столкнулись с проблемой, когда в момент инициации сообщения нам не известен номер телефона. Например, у нас есть только UUID пользователя, по которому можно вытащить телефон. А в момент обработки, мы можем сходить в базу и получить телефон. Под бэком я имею ввиду два разных laravel приложений.

Это кейс, когда 2 разных бэкенда со своими бд. Но только в одной лежит номер телефона. Инициация сообщения происходит из одного бэка, а обработчик запущен на втором.

Я понимаю, что такое происходит достаточно редко. В либе для FCM мы сразу решили этот вопрос, так как там было это предусмотрено.

@jhaoda
Copy link
Member

jhaoda commented Mar 18, 2020

@kepka42 не вижу в либе https://github.com/laravel-notification-channels/fcm такой возможности.

@kepka42
Copy link
Author

kepka42 commented Mar 20, 2020

@jhaoda https://github.com/benwilkins/laravel-fcm-notification в этой либе есть

@kepka42 kepka42 closed this as completed Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants