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

fix: Force the update of the current konnector (SCR-682) #2666

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

doubleface
Copy link
Contributor

To be sure to have updated konnector fields displayed to the user.
This is a step backward regarding time optimization but we need to be
sure to have updated konnector fields or this can lead to block
konnector errors.

To be sure to have updated konnector fields displayed to the user.
This is a step backward regarding time optimization but we need to be
sure to have updated konnector fields or this can lead to block
konnector errors.
Copy link
Contributor

@JF-Cozy JF-Cozy Dec 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'ai l'impression qu'on change le fonctionnement ici. Avant on récupérait un konnector en prop qu'on stockait en state et on settait les triggers associés. Maintenant il y a des cas où les deux sont dissociés 🤔

Est-ce qu'on ne ferait pas plutôt un useEffect dédié pour récupérer konnector = await getKonnector(client, slug) et renommer load en getTriggers par exemple ?

Ou mieux, ne faut-il pas récupérer konnector = await getKonnector(client, slug) avant et le passer à useKonnectorWithTriggers finalement ?

Et dans useKonnectorWithTriggers peut-être faut-il setter le state konnector avec la prop injectedKonnector, ou ne pas utiliser de state mais une const... ou utiliser un useLayoutEffect 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avant, dans le cas où injectedKonnector n'était pas initialisé, konnector et triggers était dissociés puisque provenant de deux requêtes différentes. Je n'ai pas l'impression que ce soit un changement fondamental.

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.

2 participants