Closed VinicSilva closed 6 years ago
Grande @VinicSilva ! Vamos as observações da sua PR:
Tem uma migration que tá "embarreirando", pois ela é a mais recente e tá tentando criar a tabela users
. Remove essa migration Version20180702012034.php
que resolve o problema;
Diminua o tamanho dos campos string/varchar, estão todos como VARCHAR(255)
;
Os campos que são sinalizados como FLAGS
coloque-os como char(1)
e batize de forma coerente com um caractere que represente-a;
A coluna partnerts.created_by
está como DATETIME, ele deve ser um int
e ter um relacionamento com users
;
Na action show
, pode colocar as informações: updated_at
e created_by
;
Tinha um parceiro com foto e quando fui alterar somente o nome ele não preservou a foto e removeu;
Os campos de Slug não devem ficar no formulário, eles devem ser populados em tempo de execução. Adiciona o pacote stof/doctrine-extensions-bundle
, importa o namespace Gedmo\Mapping\Annotation as Gedmo
e anota o campo de slug @Gedmo\Slug(fields={"name"}, updatable=true)
Na edição do parceiro, exibe a foto atual do registro para que o usuário veja qual a imagem atual;
Não tá sendo feito a validação dos campos nem no FRONTEND e nem no BACKEND. Tenta criar um novo parceiro não preenchendo nenhum campo;
Na action index
(tela de listagem) coloca um filtro por nome
e pelo tipo
do parceiro;
Vamos convencionar a estrutura de pasta de uploads assim: /public/uploads/{nome_do_modulo}
. Nessa task ficaria: /public/uploads/parceiros
;
Remove os comentários de código desnecessários do PartnerRepository
gerados pelo make
;
Foram feitas as correções, ficou faltando somente o tratamento da logo no editar. Vou pedir a ajuda de vocês para ver a melhor forma.
Show de bola! Já já irei olhar novamente ;)
@VinicSilva acabei de revisar as suas alterações e tenho apenas alguns pequenos refinamentos:
A imagem do parceiro tem que ser obrigatório tanto na sua tabela como no formulário;
Nos filtros dos parceiro, no <select>
de tipo, deve vir inicialmente com um <option value="">
com o texto "Todos os Tipos". Só deverá ser filtrado por tipo por intervenção do usuário;
O mesmo <option value="">
deve ser colocado no formulário de cadastro do Parceiro;
Mas creio que sejam apenas refinos, fora isso que esteja apto a ser mergeado. @Luitame @wilcorrea @alinechavessilva dêem uma olhada.
@VinicSilva seguindo a documentação basta por como padrão no PartnerType
o label "Selecione", e lá na página de consulta tu coloca o label conforme a última observação, assim:
<div class="col-md-3 form-group">
{{ form_label(form.type, "Todos os Tipos") }}
{{ form_widget(form.type, {"attr": {"class": "form-control"}}) }}
{{ form_errors(form.type) }}
</div>
Depois de uma longa conversa hein @VinicSilva , vou mergear agora! Parabéns e obrigado pela contribuição mano! =)
Foi colocado a criação de caminhos para uploads de imagens dentro de /public em uma variável global. No CRUD de parceiros é possivel efetuar além do preenchimento dos campos de texto, efetuar também o upload da logo do parceiro. Ficou pendente apenas o uso da paginação, vou precisar da dica de vocês para qual utilizar.