Skip to content

Fs_Challenge/ views and migrations #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

kinhosz
Copy link

@kinhosz kinhosz commented Nov 19, 2021

Views and Migrations

Construção da tela obedecendo o estilo da página e migrations necessárias para o banco de dados.

O que foi feito?

Telas para a entidade user

  • List
  • Edit
  • View
  • New

Atributos adicionados:

  • author para books
  • return date para Lends
  • name para user
  • user reference para Lends

Validations implementadas:

  • Um user só pode ter no máximo dois empréstimos de livros registrados sem entrega.
  • datetimes válidos e com intervalo de tempo menor que 6 meses em um empréstimo.

🛠 Bugs

Após implementar as validações, alguns erros são visíveis, tais como:

  • Ao adicionar mais um empréstimo, o erro de "livro não existe" aparece.
  • Ao destruir uma entidade, ela ainda continua renderizada na tela.
  • Rails não consegue encontrar alguns dados via ID.

Uma nova PR subirá e, breve com o fix para estes bugs apresentados.

Todos os Bugs foram resolvidos.

@kinhosz kinhosz added the bug Something isn't working label Nov 19, 2021
@kinhosz
Copy link
Author

kinhosz commented Nov 20, 2021

Pessoal, descobri um erro maior que está gerando outros erros.

Meu webpacker não está criando o manifest.json no public/packs/...

A aplicação só consegue rodar pq comentei a referência no view do application.html, mas isso acaba sendo ruim, já que não consigo executar o delete, por exemplo.

@kinhosz
Copy link
Author

kinhosz commented Nov 20, 2021

O erro era na versão que o meu node estava. Consegui gerar os packs e já posso prosseguir (y)

@kinhosz
Copy link
Author

kinhosz commented Nov 21, 2021

Todos os erros consertados e desafio concluído. Falta apenas subir no heroku :alegria:

@kinhosz kinhosz added enhancement New feature or request good first issue Good for newcomers and removed bug Something isn't working labels Nov 21, 2021
user = User.find_by!(email: lend_params[:user_email])
@lend.book_id = lend_params[:book_id]
@lend.user_id = user.id
end
Copy link
Contributor

@thiagodiniz thiagodiniz Nov 24, 2021

Choose a reason for hiding this comment

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

Como vocês faria isso utilizando uma abordagem de serviço?

Outra coisa, não vamos poder mergiar isso nesse repo, por que é dos proximos candidatos 🙄

@@ -0,0 +1,7 @@
require "test_helper"
Copy link
Contributor

Choose a reason for hiding this comment

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

é um otimo exercicio fazer testes, minha sugestão use o RSpec no projeto que é o que usamos nas nossas aplicações

Copy link
Contributor

@thiagodiniz thiagodiniz left a comment

Choose a reason for hiding this comment

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

Comentarios no código

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants