mirror of
https://github.com/crater-invoice/crater.git
synced 2025-10-27 19:51:09 -04:00
Merge pull request #152 from mdpoulter/email-subject
More descriptive email fields.
This commit is contained in:
@ -168,10 +168,6 @@ class EstimatesController extends Controller
|
||||
$data['user'] = User::find($userId)->toArray();
|
||||
$data['company'] = Company::find($estimate->company_id);
|
||||
$email = $data['user']['email'];
|
||||
$notificationEmail = CompanySetting::getSetting(
|
||||
'notification_email',
|
||||
$request->header('company')
|
||||
);
|
||||
|
||||
if (!$email) {
|
||||
return response()->json([
|
||||
@ -179,13 +175,7 @@ class EstimatesController extends Controller
|
||||
]);
|
||||
}
|
||||
|
||||
if (!$notificationEmail) {
|
||||
return response()->json([
|
||||
'error' => 'notification_email_does_not_exist'
|
||||
]);
|
||||
}
|
||||
|
||||
\Mail::to($email)->send(new EstimatePdf($data, $notificationEmail));
|
||||
\Mail::to($email)->send(new EstimatePdf($data));
|
||||
}
|
||||
|
||||
$estimate = Estimate::with([
|
||||
@ -340,10 +330,6 @@ class EstimatesController extends Controller
|
||||
$data['company'] = Company::find($estimate->company_id);
|
||||
|
||||
$email = $data['user']['email'];
|
||||
$notificationEmail = CompanySetting::getSetting(
|
||||
'notification_email',
|
||||
$request->header('company')
|
||||
);
|
||||
|
||||
if (!$email) {
|
||||
return response()->json([
|
||||
@ -351,13 +337,7 @@ class EstimatesController extends Controller
|
||||
]);
|
||||
}
|
||||
|
||||
if (!$notificationEmail) {
|
||||
return response()->json([
|
||||
'error' => 'notification_email_does_not_exist'
|
||||
]);
|
||||
}
|
||||
|
||||
\Mail::to($email)->send(new EstimatePdf($data, $notificationEmail));
|
||||
\Mail::to($email)->send(new EstimatePdf($data));
|
||||
|
||||
if ($estimate->status == Estimate::STATUS_DRAFT) {
|
||||
$estimate->status = Estimate::STATUS_SENT;
|
||||
|
||||
@ -12,7 +12,7 @@ use Crater\Invoice;
|
||||
use Crater\InvoiceItem;
|
||||
use Carbon\Carbon;
|
||||
use Crater\Item;
|
||||
use Crater\Mail\invoicePdf;
|
||||
use Crater\Mail\InvoicePdf;
|
||||
use function MongoDB\BSON\toJSON;
|
||||
use Illuminate\Support\Facades\Log;
|
||||
use Crater\User;
|
||||
@ -167,11 +167,6 @@ class InvoicesController extends Controller
|
||||
$data['user'] = User::find($request->user_id)->toArray();
|
||||
$data['company'] = Company::find($invoice->company_id);
|
||||
|
||||
$notificationEmail = CompanySetting::getSetting(
|
||||
'notification_email',
|
||||
$request->header('company')
|
||||
);
|
||||
|
||||
$email = $data['user']['email'];
|
||||
|
||||
if (!$email) {
|
||||
@ -180,13 +175,7 @@ class InvoicesController extends Controller
|
||||
]);
|
||||
}
|
||||
|
||||
if (!$notificationEmail) {
|
||||
return response()->json([
|
||||
'error' => 'notification_email_does_not_exist'
|
||||
]);
|
||||
}
|
||||
|
||||
\Mail::to($email)->send(new invoicePdf($data, $notificationEmail));
|
||||
\Mail::to($email)->send(new InvoicePdf($data));
|
||||
}
|
||||
|
||||
$invoice = Invoice::with(['items', 'user', 'invoiceTemplate', 'taxes'])->find($invoice->id);
|
||||
@ -408,10 +397,6 @@ class InvoicesController extends Controller
|
||||
$data['user'] = User::find($userId)->toArray();
|
||||
$data['company'] = Company::find($invoice->company_id);
|
||||
$email = $data['user']['email'];
|
||||
$notificationEmail = CompanySetting::getSetting(
|
||||
'notification_email',
|
||||
$request->header('company')
|
||||
);
|
||||
|
||||
if (!$email) {
|
||||
return response()->json([
|
||||
@ -419,13 +404,7 @@ class InvoicesController extends Controller
|
||||
]);
|
||||
}
|
||||
|
||||
if (!$notificationEmail) {
|
||||
return response()->json([
|
||||
'error' => 'notification_email_does_not_exist'
|
||||
]);
|
||||
}
|
||||
|
||||
\Mail::to($email)->send(new invoicePdf($data, $notificationEmail));
|
||||
\Mail::to($email)->send(new InvoicePdf($data));
|
||||
|
||||
if ($invoice->status == Invoice::STATUS_DRAFT) {
|
||||
$invoice->status = Invoice::STATUS_SENT;
|
||||
|
||||
@ -305,10 +305,6 @@ class PaymentController extends Controller
|
||||
$data['user'] = User::find($userId)->toArray();
|
||||
$data['company'] = Company::find($payment->company_id);
|
||||
$email = $data['user']['email'];
|
||||
$notificationEmail = CompanySetting::getSetting(
|
||||
'notification_email',
|
||||
$request->header('company')
|
||||
);
|
||||
|
||||
if (!$email) {
|
||||
return response()->json([
|
||||
@ -316,13 +312,7 @@ class PaymentController extends Controller
|
||||
]);
|
||||
}
|
||||
|
||||
if (!$notificationEmail) {
|
||||
return response()->json([
|
||||
'error' => 'notification_email_does_not_exist'
|
||||
]);
|
||||
}
|
||||
|
||||
\Mail::to($email)->send(new PaymentPdf($data, $notificationEmail));
|
||||
\Mail::to($email)->send(new PaymentPdf($data));
|
||||
|
||||
return response()->json([
|
||||
'success' => true
|
||||
|
||||
@ -12,17 +12,14 @@ class EstimatePdf extends Mailable
|
||||
|
||||
public $data = [];
|
||||
|
||||
public $notificationEmail = '';
|
||||
|
||||
/**
|
||||
* Create a new message instance.
|
||||
*
|
||||
* @return void
|
||||
*/
|
||||
public function __construct($data, $notificationEmail)
|
||||
public function __construct($data)
|
||||
{
|
||||
$this->data = $data;
|
||||
$this->notificationEmail = $notificationEmail;
|
||||
}
|
||||
|
||||
/**
|
||||
@ -32,6 +29,9 @@ class EstimatePdf extends Mailable
|
||||
*/
|
||||
public function build()
|
||||
{
|
||||
return $this->from($this->notificationEmail)->markdown('emails.send.estimate', ['data', $this->data]);
|
||||
$company = $this->data['company']['name'];
|
||||
|
||||
return $this->subject("Estimate from $company")
|
||||
->markdown('emails.send.estimate', ['data', $this->data]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -31,6 +31,8 @@ class EstimateViewed extends Mailable
|
||||
public function build()
|
||||
{
|
||||
$email = $this->data['user']['email'];
|
||||
return $this->from($email)->markdown('emails.viewed.estimate', ['data', $this->data]);
|
||||
$name = $this->data['user']['name'];
|
||||
return $this->from($email, $name)
|
||||
->markdown('emails.viewed.estimate', ['data', $this->data]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -6,23 +6,20 @@ use Illuminate\Mail\Mailable;
|
||||
use Illuminate\Queue\SerializesModels;
|
||||
use Illuminate\Contracts\Queue\ShouldQueue;
|
||||
|
||||
class invoicePdf extends Mailable
|
||||
class InvoicePdf extends Mailable
|
||||
{
|
||||
use Queueable, SerializesModels;
|
||||
|
||||
public $data = [];
|
||||
|
||||
public $notificationEmail = '';
|
||||
|
||||
/**
|
||||
* Create a new message instance.
|
||||
*
|
||||
* @return void
|
||||
*/
|
||||
public function __construct($data, $notificationEmail)
|
||||
public function __construct($data)
|
||||
{
|
||||
$this->data = $data;
|
||||
$this->notificationEmail = $notificationEmail;
|
||||
}
|
||||
|
||||
/**
|
||||
@ -32,6 +29,9 @@ class invoicePdf extends Mailable
|
||||
*/
|
||||
public function build()
|
||||
{
|
||||
return $this->from($this->notificationEmail)->markdown('emails.send.invoice', ['data', $this->data]);
|
||||
$company = $this->data['company']['name'];
|
||||
|
||||
return $this->subject("Invoice from $company")
|
||||
->markdown('emails.send.invoice', ['data', $this->data]);
|
||||
}
|
||||
}
|
||||
@ -31,6 +31,8 @@ class InvoiceViewed extends Mailable
|
||||
public function build()
|
||||
{
|
||||
$email = $this->data['user']['email'];
|
||||
return $this->from($email)->markdown('emails.viewed.invoice', ['data', $this->data]);
|
||||
$name = $this->data['user']['name'];
|
||||
return $this->from($email, $name)
|
||||
->markdown('emails.viewed.invoice', ['data', $this->data]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -13,17 +13,14 @@ class PaymentPdf extends Mailable
|
||||
|
||||
public $data = [];
|
||||
|
||||
public $notificationEmail = '';
|
||||
|
||||
/**
|
||||
* Create a new message instance.
|
||||
*
|
||||
* @return void
|
||||
*/
|
||||
public function __construct($data, $notificationEmail)
|
||||
public function __construct($data)
|
||||
{
|
||||
$this->data = $data;
|
||||
$this->notificationEmail = $notificationEmail;
|
||||
}
|
||||
|
||||
/**
|
||||
@ -33,6 +30,9 @@ class PaymentPdf extends Mailable
|
||||
*/
|
||||
public function build()
|
||||
{
|
||||
return $this->from($this->notificationEmail)->markdown('emails.send.payment', ['data', $this->data]);
|
||||
$company = $this->data['company']['name'];
|
||||
|
||||
return $this->subject("Payment from $company")
|
||||
->markdown('emails.send.payment', ['data', $this->data]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -1,6 +1,6 @@
|
||||
@component('mail::message')
|
||||
# Introduction
|
||||
Customer viewed this Estimate.
|
||||
{{ $data['user']['name'] }} viewed this Estimate.
|
||||
|
||||
@component('mail::button', ['url' => url('/admin/estimates/'.$data['estimate']['id'].'/view')])
|
||||
Estimate
|
||||
|
||||
@ -1,6 +1,6 @@
|
||||
@component('mail::message')
|
||||
# Introduction
|
||||
Customer viewed this Invoice.
|
||||
{{ $data['user']['name'] }} viewed this Invoice.
|
||||
|
||||
@component('mail::button', ['url' => url('/admin/invoices/'.$data['invoice']['id'].'/view')])
|
||||
Invoice
|
||||
|
||||
Reference in New Issue
Block a user