Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

697 add 1n relation between ads and media #914

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ivanlegranbizarro
Copy link
Collaborator

Kevin, hay un par de schemas que fallan. Esto es solo para que le eches un ojo al script generador. En teoría ahora se reflejan las relaciones en los schemas de zod y se autoimportan las dependencias (más o menos).

@ivanlegranbizarro ivanlegranbizarro linked an issue Mar 16, 2023 that may be closed by this pull request
@kevinmamaqi
Copy link
Contributor

Kevin, hay un par de schemas que fallan. Esto es solo para que le eches un ojo al script generador. En teoría ahora se reflejan las relaciones en los schemas de zod y se autoimportan las dependencias (más o menos).

Buen trabajo, ¿Has comprobado que parecen ser schemas requeridos? Quizás deberían de ser opcionales.

@ivanlegranbizarro
Copy link
Collaborator Author

ivanlegranbizarro commented Mar 17, 2023

Hola, he estado trabajando más en el script. Hice lo que me sugeriste de que las relaciones fueran opcionales (creo, si es que no se me ha escapado nada) y he arreglado los imports, porque por ejemplo InvoiceSchema no importaba el Status del PrismaClient. Además, LevelSchema se autoimportaba a sí mismo en LevelSchema.js con la solución que estaba proponiendo antes. Además he añadido lo de lazy para que no haya problemas circulares.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add 1:N relation between Ads and Media
2 participants