[CA-4747] Profile editor should update consents with key in snakecase #280
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
moust
commented
Apr 7, 2025
Comment on lines
+138
to
+154
| /** | ||
| * @todo this can be removed when https://github.com/ReachFive/identity-web-core-sdk/pull/260 will be merged | ||
| * L'idéal serait que la clé d'un consentement soit en valeur d'une propriété "key" et non une clé de la structure Map utilisée. | ||
| * le mieux serait même que `consents` soit un array et non un map. | ||
| * Mais bon, difficile de changer ça maintenant sans créer un breaking change pour les utilisateurs de l'api ou du sdk core... | ||
| */ | ||
| .then(({ consents, ...profile }) => ({ | ||
| ...profile, | ||
| consents: consents | ||
| ? Object.fromEntries( | ||
| Object.entries(consents).map(([key, value]) => [ | ||
| key, | ||
| camelCaseProperties(value) as UserConsent | ||
| ]) | ||
| ) | ||
| : undefined | ||
| }) as Profile) /** @todo check api response key format in sdk core */ |
There was a problem hiding this comment.
Cette partie sert juste à palier au fait que le SDK Core format la réponse api en camelcase mais en faisant exception des concents et de toute la structure enfant.
J'ai fait une PR sur le SDK Core pour que seul la clé d'un consentement soit préservée en snakecase. Mais j'ai peur que ça introduise un breaking change pour d'éventuelles intégrations avec le SDK Core...
moust
commented
Apr 7, 2025
| @@ -80,6 +83,7 @@ export default function consentField({ | |||
| ...props, | |||
| required, | |||
| defaultValue: { granted: props.defaultValue ?? false }, | |||
| mapping: new PathMapping(snakeCasePath(props.path ?? props.key)), // Consent key should be snake_case | |||
There was a problem hiding this comment.
Le coeur du problème. PathMapping sert à faire la mapping entre le modèles et les champs d'un formulaire. Par défaut ça fait une conversion camelcase du le nom des propriétés...
(les joies de l'usine à gaz que sont les formulaire sur le SDK UI...)
thomasreach5
approved these changes
Apr 8, 2025
sachaMemmir5
approved these changes
Apr 9, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
CA-4747
L'idéal serait que la clé d'un consentement soit en valeur d'une propriété "key" et non une clé de la structure Map utilisée.
Le mieux serait même que
consentssoit un array et non un map.Mais bon, difficile de changer ça maintenant sans créer un breaking change pour les utilisateurs de l'api ou du sdk core...