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

Fixing up the notification is not being sent for the user that has been added from the filamentphp dashboard #118

Open
Eng3bbas opened this issue Jun 18, 2024 · 6 comments
Assignees

Comments

@Eng3bbas
Copy link

Eng3bbas commented Jun 18, 2024

//fixing up creation token not created and the notification not sended
protected function mutateFormDataBeforeCreate(array $data): array {
$data['type'] = "db"; return parent::mutateFormDataBeforeCreate($data); }

@mihaisolomon
Copy link
Collaborator

Please create a pull request with this fix.

@mtnkosi
Copy link

mtnkosi commented Jul 14, 2024

I installed Helper on Firday (12 July 2024) on my local machine. Everything looks to be fine except for sending a notifaction when I create new user. What must I do to get this working?

@mtnkosi
Copy link

mtnkosi commented Jul 17, 2024

I installed Helper on Firday (12 July 2024) on my local machine. Everything looks to be fine except for sending a notifaction when I create new user. What must I do to get this working?

I had to make few changes to get this working. I am now sorted - the notification is working.

@mihaisolomon
Copy link
Collaborator

Can you make a PR with the changes that you did, so we can fix the problem?

@mtnkosi
Copy link

mtnkosi commented Jul 17, 2024

Can you make a PR with the changes that you did, so we can fix the problem?

My fix is almost similar to what AbbasovMakrov did. If he hasn't created a pull request, I can create one.

@YongkyA23
Copy link

where do i need to put this fix? thx in advance

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

4 participants