arlissonjaques/butterfly-api

Uma API simples para uma custureira expor seus trabalhos e receber pedidos através do Telegram.

butterfly-api's Issues

Enviando contato vazio

Problema:

• Quando alguns dos parâmetros é invalido, o contato não é criado no banco, mas é enviado ao Telegram.

Como deve ser:

• Só deve ser feito contato após ser salvo no banco, caso não seja salvo, nada deve ser enviado ao Telegram.

Retirar linhas

Nos controllers existe uns separadores:

#------------------------------------------------------

Por favor, retirar de todos.

Importação de arquivo em controller

Atualmente, existe uma importação de arquivo dentro de um controller.

require './lib/integration/contact/contact_bot'

Possível solução, dentro de /config/application.rb

config.eager_load_paths << Rails.root.join('lib/pasta_alea')

Múltiplos contatos por usuário

Problema:

• Só está podendo ser feito um contato por e-mail. Atualmente caso o usuário queira fazer mais de um contato, terá que usar emails diferentes.

Como deve ser:

• O usuário deve poder fazer quantos contatos quiser usando o mesmo e-mail.

Adicionar imagens para os produtos

Como esperado, é necessario que os produtos tenham imagens, pois é o que atrairá os clientes. É interessante que as imagens sejam subidas para a S3.

Erro em category_controller action update

Em category_controller algumas coisas inesperadas estão acontecendo.

Quando é feita a requisição:

PUT "/api/v1/categories/:id" sem nenhum paramentro.

Está sendo retornada o status 200, e é retornada a categoria referente ao :id. Ex:

{
  "id": 2,
  "name": "calcados",
  "created_at": "2021-06-07T20:11:54.037Z",
  "updated_at": "2021-06-07T20:17:28.236Z"
}

Porém, ao meu ver o comportamento correto deveria ser retornar um erro do tipo 422, visto que a requisição foi sem nenhum paramento. Em category existe apenas o paramento :name, então caso este parametro não fosse submetido deveria ser retornado o erro, porque só existe :name para alterar.

Na linha 45 exite o filtro params.permit(:name, :product_id), talvez uma outra abordagem resolva o problema.

Por favor estudar o caso e encontrar a solução.

Remover actions desnecessarias

Em contacts_controller existem algumas ações que não são interessantes para o momento.

index - todas as os contatos serão visto através do telegram.
update - o cliente não poderá editar um contato já enviado, precisará enviar um novo.
destroy - o cliente não poderá excluir um contato já enviado.

Apenas create é necessário.

Adicionar testes faltando

Projeto ainda está sem testes, o quanto antes adicionar:

• Request
  • category
  • user
  • contact
  • product
• Lib
• Controllers?

Remoção de ação

Dentro de contacts_controller existe uma ação show que não está sendo utilizada, seria interessante remove-la.

Adicionar dados sensíveis em uma variável de ambiente

Atualmente alguns dados confidencias podem ser acessados facilmente sem qualquer restrição. É importando colocar esses dados dentro de um .ENV.

Paramento desnecessario talvez?

Em app/controller/api/v1/categories_controller.rb na linha 43, existe um paramentro que não me recordo se ele é realmente necessario. Estude o contexto e caso não seja necessario remova-o. Caso seja, comente aqui na issue o porquê.