8000 Fixing up the notification is not being sent for the user that has been added from the filamentphp dashboard · Issue #118 · devaslanphp/project-management · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
Open
Eng3bbas opened this issue Jun 18, 2024 · 6 comments
Open

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

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

Comments

@Eng3bbas
Copy link
Eng3bbas commented Jun 18, 2024 •
edited
Loading

//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 •
edited
Loading

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
Assignees

@Eng3bbas Eng3bbas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mihaisolomon @Eng3bbas @YongkyA23 @mtnkosi
0