arlissonjaques/butterfly-api

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

butterfly-api's Issues

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.

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.

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')

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ê.

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.

Retirar linhas

Nos controllers existe uns separadores:

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

Por favor, retirar de todos.

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.

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.

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.

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.