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(geo): check catalog not undefined before loading items #1583

Merged
merged 5 commits into from
Jan 19, 2024

Conversation

aziz-access
Copy link
Contributor

@aziz-access aziz-access commented Jan 16, 2024

What is the current behavior? (You can also link to an open issue here)

this behavior related ti #1573

What is the new behavior?

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Other information:

@aziz-access aziz-access added the bug Incorrect or unexpected behaviors // Anomalies de fonctionnement label Jan 16, 2024
@aziz-access aziz-access self-assigned this Jan 16, 2024
@aziz-access aziz-access requested review from alecarn and pelord January 16, 2024 17:56
this.loadCatalogItems(this.catalog);
if (this.catalog) {
this.loadCatalogItems(this.catalog);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

On ne semble pas régler le problème mais plutot éviter de la renvoyer. Rapidement ça me semble être un problème de concurence entre les deux observables. Il faudrait trouver un moyen de corriger ce problème

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui exactement, problème de concurrence, parfois this.catalog est undefined.

@aziz-access aziz-access requested a review from alecarn January 17, 2024 19:48
@alecarn alecarn removed the request for review from pelord January 18, 2024 14:19
@alecarn alecarn requested a review from pelord January 18, 2024 22:39
@alecarn
Copy link
Collaborator

alecarn commented Jan 18, 2024

@pelord Ready for review

@pelord pelord changed the title fix(geo): check catalog not undefined before loading iteams fix(geo): check catalog not undefined before loading items Jan 18, 2024
@alecarn alecarn merged commit 1efb4a6 into next Jan 19, 2024
4 checks passed
@alecarn alecarn deleted the 1573-check-catalog-befor-get-items branch January 19, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect or unexpected behaviors // Anomalies de fonctionnement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError - Integration/CatalogState Cannot read properties of undefined (reading 'id')
3 participants