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

Add table t_meta_taxref #403

Merged
merged 3 commits into from
Jul 11, 2023
Merged

Add table t_meta_taxref #403

merged 3 commits into from
Jul 11, 2023

Conversation

amandine-sahl
Copy link
Contributor

fix : #394

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Jun 24, 2023

Le max_cd_nom ne va fonctionner que si aucun taxon maison n'a été ajouté dans le Taxref, non ?
C'est un cas fréquent.
Mais sinon on dira dans les notes de version de vérifier ou compléter le champs. Fait dans 4767a68

@amandine-sahl
Copy link
Contributor Author

amandine-sahl commented Jun 26, 2023

Le max_cd_nom ne va fonctionner que si aucun taxon maison n'a été ajouté dans le Taxref, non ?

L'alias s'appelle comme ça car il correspond à la valeur maximale du cd_nom pour une version de taxref. La requête ne cherche pas la valeur max du cd_nom dans la table taxref, mais si la valeur maximale pour le taxref donné est dans la table taxref.

Mais c'est bien d'indiquer de vérifier dans la note de version

@camillemonchicourt
Copy link
Member

Le max_cd_nom ne va fonctionner que si aucun taxon maison n'a été ajouté dans le Taxref, non ?

L'alias s'appelle comme ça car il correspond à la valeur maximale du cd_nom pour une version de taxref. La requête ne cherche pas la valeur max du cd_nom dans la table taxref, mais si la valeur maximale pour le taxref donné est dans la table taxref.

Mais c'est bien d'indiquer de vérifier dans la note de version

OK je vois, ça fonctionnera alors.
Mais oui j'ai indiqué de vérifier.

@TheoLechemia TheoLechemia force-pushed the feat/add_metadatatable branch 2 times, most recently from fe745eb to 22ad58c Compare July 11, 2023 11:59
@camillemonchicourt camillemonchicourt merged commit bb4a571 into develop Jul 11, 2023
6 checks passed
@camillemonchicourt camillemonchicourt deleted the feat/add_metadatatable branch July 11, 2023 12:26
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.

3 participants