Skip to content

fix(certificateParentMerger): append child externalContents to parent ones instead of overriding (ARI-3292)#490

Open
stefdelec wants to merge 1 commit into
masterfrom
fix/ari-3292-external-contents
Open

fix(certificateParentMerger): append child externalContents to parent ones instead of overriding (ARI-3292)#490
stefdelec wants to merge 1 commit into
masterfrom
fix/ari-3292-external-contents

Conversation

@stefdelec

Copy link
Copy Markdown
Member

Quoi

certificateParentMerger fusionnait parent/enfant avec mergeWith sans customizer : les tableaux étaient fusionnés index par index, donc le 1er externalContents de l'enfant écrasait la notice du parent. Désormais les externalContents sont concaténés (parent d'abord, dédoublonnage des entrées identiques).

Pendant côté SDK legacy du fix backend ArianeeBrandDataHub#3846 (mêmes corrections dans wallet-api#133 et arianee-sdk#421).

⚠️ Repo dormant : dernier commit septembre 2024, aucun commit depuis 12 mois. Le package npm @arianee/arianeejs (1.119.0) est toujours publié et non déprécié, d'où ce correctif pour les intégrations legacy. À merger seulement si le repo est encore considéré comme supporté — sinon, à fermer.

Comment

mergeWith reçoit désormais un customizer (lodash déjà utilisé) :

  • externalContents concaténés (parent d'abord, dédoublonnage uniqWith(isEqual)) ;
  • i18n mergé par langue (langues parent seules conservées, langues enfant seules ajoutées), avec concaténation des externalContents par langue ;
  • les autres clés gardent le comportement de merge existant.

Tests

  • Le test existant (it should merge in order parents) reste vert.
  • 4 nouveaux cas : concat, parent seul, dédoublonnage, i18n par langue. Couverture 100% du fichier.

Ticket : ARI-3292

🤖 Generated with Claude Code

… ones instead of overriding (ARI-3292)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

1 participant