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

Refs niet geladen #40

Open
johannesdewit opened this issue Aug 25, 2023 · 2 comments
Open

Refs niet geladen #40

johannesdewit opened this issue Aug 25, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@johannesdewit
Copy link
Collaborator

Ik had het lemma ‘Allemannischen Gedichte’ veranderd in ‘Allemannische Gedichte’ maar niet bij de children (de lemmata met ‘Allemannischen Gedichte’ als parent). De index werd wel gedeployed, maar zonder enige referenties onder de lemmata. Dit lijkt me niet de bedoeling.

Het zou mooi zijn als we zouden kunnen valideren dat de parents ook bestaan, zowel in het validatieschema als in populate_index. Dat voorkomt dat we een lege index online hebben.

@johannesdewit johannesdewit added the bug Something isn't working label Aug 25, 2023
@andredelft
Copy link
Owner

Klinkt alsof er inderdaad geen goede check is ingebouwd voor het geval dat een lemma verwijst naar een parent die niet bestaat, waardoor populate_index faalt. Ik heb bij het toevoegen van een parent-child-relatie via de python functie wel zo'n check ingebouwd, maar ben er vervolgens denk ik vanuit gegaan dat deze verwijzingen in de index altijd bestaan.

Dat moeten we dus sowieso even toevoegen. Validatie zou mooi zijn, maar ik denk eigenlijk niet dat zoiets kan met zo'n schema. Maar op zich weerhoudt ook niets ons ervan om extra tests toe te voegen aan de validatie, met losse python scripts.

@johannesdewit
Copy link
Collaborator Author

Ik geloof inderdaad niet dat je dat kan doen met het schema, aangezien het om meer gaat dan alleen syntaxis. Met losse pythonscripts moet het sowieso kunnen, een aantal test erbij en klaar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants