Fórum Academy - E-mail Laravel

Criador Brendo Vale 03/10/2018

Olá na minha notificação eu tenho os seguintes códigos para envio de email:

return (new MailMessage)

->subject('Verificação de e-mail - MVPSystem')

->line('Por favor, clique no botão abaixo para verificar o seu endereço de email.')

->action('Verificar o endereço de email',$this->verificationUrl($notifiable))

->line('Se você não criou uma conta, nenhuma ação adicional é necessária.');

-------------

como eu faço pra ler no arquivo blade os itens: subject, line, action, line. ?

Eu entendi aqui https://blog.especializati.com.br/envio-de-e-mails-no-laravel/ mas n sei como fazer a leitura desses itens :(

16 comentários

Criador Brendo Vale 03/10/2018

Obs: eu não quero utilizar os componentes padrões de e-mail do laravel. Quero customizar o estilo completo para cada notificação

Manager Carlos Ferreira 03/10/2018

Olá, Brendo!
Tudo bem?

Você finalizou esse ticket.
Conseguiu resolver? Tudo certo?

Criador Brendo Vale 03/10/2018

Mais ou menos, consegui gerar um estilo mas é muito confuso... tudo em uma só página...

tem como eu te mandar meu projeto de estudo por email pra vc dar uma corrigida pra mim?

Criador Brendo Vale 03/10/2018

Digo, ver se as logicas do sistema está correta e tals... usei o curso de acl e fiz outras configurações sozinho então não sei se é gambiarra ou se está certo... outro problema é que está usando 10mb de ram... mt estranho

Manager Carlos Ferreira 03/10/2018

Fala Brendo!

Sobe o projeto para o GitHub, ou Bitbucket que dou uma analisada rápida e passo um feedback.

Criador Brendo Vale 03/10/2018

Sobe o projeto para o GitHub, ou Bitbucket que dou uma analisada rápida e passo um feedback.

n sei subir la, pode ser no google drive?

Criador Brendo Vale 03/10/2018

https://github.com/TioLee/Teste Ta ai o link do github

Criador Brendo Vale 03/10/2018

correção : https://github.com/TioLee/Laravel-ACL-Treino

Manager Carlos Ferreira 03/10/2018

Brendo,

A lógica de envio de e-mails ficou na Notification MailVerification? É essa classe em especial que você deseja que eu analise? :)

Criador Brendo Vale 03/10/2018

Não, não to utilizando os emails, falo no codigo em geral mesmo se puder dar uma olhada pra ver se tem gambiarra rs.

Uma pergunta, como eu passo uma variavel para todas as views sem ter que ficar digitando em todos os métodos?

por exemplo, notificações... quero carregas as notificações não vistas mas tenho que por em todas os métodos :/

Criador Brendo Vale 03/10/2018

fiz assim as notificações...

//carrega as notificações vistas
$visualized = auth()->user()->notifications;
//carrega as notificações
$notificationCheck = Notification::all();
 
//percorre as notificacoes vistas e salva apenas o id na variavel notificationSystem1
$notificationSystem1 = [];
foreach($visualized as $v){
$notificationSystem1[] = $v->id;
}

//percorre as notificacoes all e salva apenas o obj nao visto na variavel notificationSystem
$notificationSystem = [];
foreach($notificationCheck as $v){
if(!in_array($v->id, $notificationSystem1))
$notificationSystem[] = $v;
}

return view('home', compact('posts', 'notificationSystem'));

Criador Brendo Vale 03/10/2018

to flodando foi mal...

Só pra explicar essa notificação ...

Many to many ai os que forem visualizados eu dou sync entre o usuario e a notificaçao

Manager Carlos Ferreira 03/10/2018

Uma das formas de compartilhar uma variável em todas as views no Laravel é usando View Composer: https://laravel.com/docs/5.7/views#view-composers
Se tiver dúvidas como resolver, me retorna que te ajudo. Ok?

Olhei rapidamente o seu código e não encontrei nenhum erro brutal ou má pratica. Você seguiu as PSRs, o código ficou bonito.
O seus controllers até que ficaram enxuto e sem lógica, +1 ponto legal;

O controller UsersController pode dá uma melhorada, enxugando ele e redistribuído as responsabilidades. O método indexDataTable ficou muito inviável.

Suas notifications ficaram OK.

Os Models OK.

Policies Ok.

SendMailUser Ok.

Criador Brendo Vale 03/10/2018

Tenho muito o que estudar... Obrigado!!

https://laravel.com/docs/5.7/views#view-composers não sabia disso e procurei muito haha

eu só não entendi essa parte : "O controller UsersController pode dá uma melhorada, enxugando ele e redistribuído as responsabilidades. O método indexDataTable ficou muito inviável."

como pensar em formas de redistribuír as responsabilidades?

Manager Carlos Ferreira 03/10/2018

Um controller deve ser o mais enxuto possível, ou seja, quanto menos código melhor!

O objetivo de um controller é delegar as responsabilidades, e não conter lógica.

O seu método indexDataTable além de ter MUITO conteúdo, ele ainda tem um erro muito grave, conteúdo HTML misturado com o PHP... Só o fato de ter html junto com o codigo já quebrou o MVC.

Criador Brendo Vale 03/10/2018

Saquei, vou corrigir obrigado!