Skip to content
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

#issue 267 - implementação da listagem de dados pet via service no componente PetAvatar #issue-267 #290

Merged
merged 6 commits into from
Aug 9, 2024

Conversation

EdiltonOliveira
Copy link
Contributor

@EdiltonOliveira EdiltonOliveira commented Aug 2, 2024

Closes #267

Feature

feat: implementação da listagem de dados pet via service no componente PetAvatar #issue-267

  • Inclusão dos dados dos Pets para mock db.json

  • Downgrade json-server 1.0.0 para 0.17.4 para personalização de rota API

  • Criação de arquivo routes.json para personalização da rota via json-server

  • Implementação do userService.json para consumo da API mockada via json-server

  • Alteração da URL de userService.js para atender a rota definida pelo BackEnd

Bugfix
  • Description
    N/A

  • Cause
    N/A

  • Solution
    N/A

Changelog N/A
Visual evidences 🖼️

petdex_front2
petdex_front1

Checklist
  • Issue linked
  • Build working correctly
  • Tests created
Additional info Para a criação de um servidor API com personalização de rota equivalente ao backend, foi necessário usar uma versão anterior do pacote json-server, visto que a versão 1.0.0 não oferecia o recurso necessário para que ao acessar a rota http://localhost:3000/api/user/1/my-pets

@PiluVitu PiluVitu self-requested a review August 2, 2024 21:28
Copy link
Contributor

@DominMFD DominMFD left a comment

Choose a reason for hiding this comment

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

Bom dia, eu to com uma dúvida, a gente já tem um mock da API, pq criar outro? é só uma dúvida mesmo até pq a implementação ta certa, mas eu to confuso

@EdiltonOliveira
Copy link
Contributor Author

Bom dia, @DominMFD . Como o mock usados para renderizar o componente PetAvatar eram diferentes dos dados db.json, optei por preservar a estrutura original de db.json e criar uma mock que melhor estivesse alinhado a tabela pet_img. Essa é o primeiro pull request, então estou aguardando feedback para ver se a decisão foi boa ou não.

Abaixo a tabela que melhor se alinha ao mock que estava sendo usado

Captura de tela de 2024-08-05 11-49-36

O mock consumido no componente PetAvatar

Captura de tela de 2024-08-05 11-56-55

Também tomei a decisão levando em conta que a API do back será disponibilizado em breve, logo criei o dbSimulatorAPI.json junto com a versão json-server 0.17 para conseguir simulador a rota e o mock fazendo o mínimo de alterações possíveis . Caso seja necessário tabela única, é provável que o json-server não consiga simular um roteamento complexo que se encaixe em db.json
Captura de tela de 2024-08-05 12-01-12

@DominMFD
Copy link
Contributor

DominMFD commented Aug 5, 2024

É, foi isso que eu pensei mesmo, vamos ver na daily hj com o Ale

@PiluVitu PiluVitu requested review from a team, Luiginicoletti, JpBurgarelli, juliaam, Frompaje and DominMFD and removed request for a team August 7, 2024 12:36
Copy link
Member

@PiluVitu PiluVitu left a comment

Choose a reason for hiding this comment

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

LGTM

src/services/userService.js Outdated Show resolved Hide resolved
@DominMFD DominMFD merged commit ae28176 into devhatt:main Aug 9, 2024
3 checks passed
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.

Vincular componente Pet Avatar com API
4 participants