Better user creation email.

Author: DaneEveritt

CHANGELOG.md

@@ -12,6 +12,7 @@ This project follows [Semantic Versioning](http://semver.org) guidelines.
 ### Changed
 * `[pre.2]` — File Manager now displays relevant information on all screen sizes, and includes better button clicking mechanics for dropdown menu.
 * Reduced the number of database queries being executed when viewing a specific server. This is done by caching the query for up to 60 minutes in memcached.
+* User creation emails include more information and are sent by the event listener rather than the repository.
 
 ## v0.6.0-pre.2 (Courageous Carniadactylus)
 ### Fixed

app/Notifications/AccountCreated.php

@@ -45,9 +45,9 @@ class AccountCreated extends Notification implements ShouldQueue
      *
      * @return void
      */
-    public function __construct($token)
+    public function __construct(array $user)
     {
-        $this->token = $token;
+        $this->user = $user;
     }
 
     /**
@@ -70,8 +70,10 @@ public function via($notifiable)
     public function toMail($notifiable)
     {
         return (new MailMessage)
-                    ->line('You are recieving this email because an account has been created for you on Pterodactyl Panel.')
-                    ->line('Email: ' . $notifiable->email)
-                    ->action('Setup Your Account', url('/auth/password/reset/' . $this->token . '?email=' . $notifiable->email));
+            ->greeting('Hello ' . $this->user->name . '!')
+            ->line('You are recieving this email because an account has been created for you on Pterodactyl Panel.')
+            ->line('Username: ' . $this->user->username)
+            ->line('Email: ' . $notifiable->email)
+            ->action('Setup Your Account', url('/auth/password/reset/' . $this->user->token . '?email=' . $notifiable->email));
     }
 }

app/Notifications/SendPasswordReset.php

@@ -75,17 +75,4 @@ public function toMail($notifiable)
             ->action('Reset Password', url('auth/password/reset', $this->token))
             ->line('If you did not request a password reset, no further action is required.');
     }
-
-    /**
-     * Get the array representation of the notification.
-     *
-     * @param  mixed  $notifiable
-     * @return array
-     */
-    public function toArray($notifiable)
-    {
-        return [
-            //
-        ];
-    }
 }

app/Observers/UserObserver.php

@@ -49,6 +49,12 @@ public function creating(User $user)
     public function created(User $user)
     {
         event(new Events\User\Created($user));
+
+        $user->notify((new AccountCreated([
+            'name' => $user->name_first,
+            'username' => $user->username,
+            'token' => DB::table('password_resets')->where('email', $user->email)->orderBy('created_at', 'desc')->first(),
+        ])));
     }
 
     /**

app/Repositories/UserRepository.php

@@ -105,8 +105,6 @@ public function create(array $data)
                     'token' => $token,
                     'created_at' => Carbon::now()->toDateTimeString(),
                 ]);
-
-                $user->notify((new AccountCreated($token)));
             }
 
             DB::commit();