-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat(taxref): add taxref_tree materialized view #567
Conversation
f89e600
to
e84cb84
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #567 +/- ##
===========================================
+ Coverage 51.98% 52.20% +0.21%
===========================================
Files 37 37
Lines 2418 2431 +13
===========================================
+ Hits 1257 1269 +12
- Misses 1161 1162 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
bc33d0c
to
242e678
Compare
242e678
to
e88bd31
Compare
PR ok pour merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testé, tout est bon de mon côté ! Juste côté changelog, autant mettre ça dans la 2.0.0 ?
Ah oui je sais pas pourquoi j’ai cru que la 2.0 était déjà sortie, je change ça |
Je pense que pour les installations classiques de TH ou GN, l'ajout de l'extension ne passera pas dans la migration, mais qu'il faudra l'installer manuellement avec une commande comme dans la 1.8.0 : https://github.com/PnX-SI/TaxHub/blob/develop/docs/changelog.md#180-2021-06-22 Mais aussi le préciser côté Changelog de GN, car c'est surtout là que les administrateurs auront à le faire. |
e88bd31
to
0d970df
Compare
The MV containes for each cd_nom, the list of all parents cd_ref.
0d970df
to
ac34c57
Compare
Pour chaque
cd_nom
, la VM contient la liste descd_ref
parents jusqu’a Biota (cd_nom 349525)Voir #354