Skip to content

Sinaliza quando o chamado possui mais de um atendente#380

Open
alecostaweb wants to merge 3 commits intouspdev:masterfrom
alecostaweb:issue_379
Open

Sinaliza quando o chamado possui mais de um atendente#380
alecostaweb wants to merge 3 commits intouspdev:masterfrom
alecostaweb:issue_379

Conversation

@alecostaweb
Copy link
Member

Fix #379

{{ Str::limit($user->name ?? '-', 20) }}
@include('chamados.show.user-detail', ['user' => $user])
{{-- @include('chamados.partials.user-', ['user' => $user]) --}}
@php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aqui não seria melhor alterar o método pessoas(pivot=null, return='first'){}

sendo o return para usar first() ou get()? Se for get pega todos.
então faria uma chamada ->pessoas('Atendente') para pegar o primeiro e outra chamada ->pessoas('Atendente','get') para pegar todos.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O método pessoas dentro da Model/Chamado?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sim

@alecostaweb
Copy link
Member Author

@masakik, acho que está ok

@alecostaweb
Copy link
Member Author

Só mais um review e podem mergear

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

Successfully merging this pull request may close these issues.

Mostrar todos os atendentes na lista meus atendimentos

2 participants