Skip to content

fix: Adjusted sql file name to reflect reality #64

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

Closed
wants to merge 1 commit into from

Conversation

w-araujo
Copy link
Contributor

🤔 O que foi feito?

Foi alterado no readme o nome do arquivo .sql para refletir o nome real do arquivo que está na pasta "prisma".

📗 Checklist do desenvolvedor

  • Foi testado localmente?
  • Foi adicionado documentação necessária (swagger, testes e etc)? Sim, no readme

👀 Checklist do revisor

Revisor 1️⃣

  • Você entendeu o propósito desse PR?
  • Você entendeu o fluxo de negócio?
  • Você entendeu o que e como foi desenvolvido tecnicamente a solução?
  • Você analisou se os testes estão cobrindo a maioria dos casos?

🔗 Referênia

@lucaskdc
Copy link

PR duplicado sobre mesmo assunto: #91

@@ -18,7 +18,7 @@ Se estiver usando Docker, os comandos para carregar o dump são:

```bash
# Copiar o dump para a pasta temporária do Docker
docker exec -it cp backup.sql POSTGRES_CONTAINER_ID:/tmp/backup.sql
docker exec -it cp dev_dump.sql POSTGRES_CONTAINER_ID:/tmp/backup.sql

Choose a reason for hiding this comment

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

this command doesn't work, as reviewd by brunousml in the PR #91

@lucaskdc
Copy link

Verificar issue #109

@gabrielbrise
Copy link

Para mudar o nome para refletir o nome atual, precisa ser alterado em ambos comandos. O PR #84 faz isso da maneira certa. Caso queira que seja usado o nome igual ao do arquivo em todas etapas do processo de dump, sugira a alteração dentro daquele PR. Não há necessidade de um PR separados mechendo nas mesmas linhas pois no fim vão gerar conflitos

Copy link

@gabrielbrise gabrielbrise left a comment

Choose a reason for hiding this comment

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

Needs changes in both commands to work. My suggestion is to close this PR and suggest changes at PR #84

@giggio
Copy link
Member

giggio commented May 20, 2024

Duplicado do #84 que já foi integrado.

@giggio giggio closed this May 20, 2024
@giggio giggio added the duplicate This issue or pull request already exists label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants