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: wait for identity success action before fetching catalog #1974

Merged

Conversation

juanmahidalgo
Copy link
Contributor

  • This PR fixes the race condition given by the fact that the getIdentity fn is not sync, so it can happen that the Fetch Items Request is triggered before getting the identity, so it gets triggered without being signed.

@juanmahidalgo juanmahidalgo merged commit 8392277 into feat/marketplace-server-catalog Aug 15, 2023
4 of 6 checks passed
@juanmahidalgo juanmahidalgo deleted the fix/identity-race-condition branch August 15, 2023 14:47
juanmahidalgo added a commit that referenced this pull request Sep 21, 2023
…#2005)

* fix: wait for identity success action before fetching catalog (#1974)

* fix: wait only if the identity is not already generated
juanmahidalgo added a commit that referenced this pull request Nov 8, 2023
…1967)

* feat: introduce logic to fetch catalog from the marketplace-server

* feat: use correct URL for the marketplace server

* feat: wait for FFs to be loaded

* fix: card overflow is clickeable

* test: fix broken tests

* feat: add generator getCatalogAPI to abstract the API selection logic

* chore: remove unwanted comment

* fix: wait for identity success action before fetching catalog (#1974)

* feat: remove old import

* test: fix broken test

* chore: reset dev.json values
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.

None yet

2 participants