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: retry window fetching #1291

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Conversation

brianp
Copy link
Collaborator

@brianp brianp commented Dec 18, 2024

Description

Seeing errors in Sentry where the main window can't be fetched. Retry for a second before panic'ing.

Motivation and Context

A brief delay between window creation and closing may help alleviate the error.

How Has This Been Tested?

Error isn't present locally so 🤞🏻

@brianp brianp requested a review from stringhandler December 18, 2024 12:46
Copy link
Contributor

@stringhandler stringhandler left a comment

Choose a reason for hiding this comment

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

Ok for now, but should really try get rid of this code altogether

@brianp brianp merged commit ba18bdd into tari-project:main Dec 18, 2024
11 checks passed
@brianp brianp deleted the fix/cant-find-window branch December 18, 2024 13:09
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