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: error notification when running onboarding tours #180

Merged

Conversation

MitanOmar
Copy link
Member

fix #144

@MitanOmar MitanOmar self-assigned this May 16, 2024
@MitanOmar MitanOmar requested a review from a team as a code owner May 16, 2024 12:41
@MitanOmar MitanOmar force-pushed the remove-error-when-tour-router-change branch 2 times, most recently from 805fa70 to f362e89 Compare May 16, 2024 13:05
@MitanOmar MitanOmar requested a review from derrabauke May 16, 2024 13:11
frontend/app/services/tour.js Outdated Show resolved Hide resolved
@c0rydoras
Copy link
Collaborator

i'd also change the commit message to something like:

fix(frontend): error notifications in onboarding tour

@MitanOmar MitanOmar force-pushed the remove-error-when-tour-router-change branch 2 times, most recently from b89e022 to 6dce422 Compare May 22, 2024 13:09
@MitanOmar MitanOmar requested a review from a team as a code owner May 22, 2024 13:09
@MitanOmar MitanOmar force-pushed the remove-error-when-tour-router-change branch 2 times, most recently from b0ae95a to c1608e4 Compare May 23, 2024 08:56
@MitanOmar MitanOmar requested review from derrabauke and c0rydoras May 23, 2024 09:38
@MitanOmar MitanOmar force-pushed the remove-error-when-tour-router-change branch from 830e825 to ad331e5 Compare May 23, 2024 09:44
@MitanOmar MitanOmar force-pushed the remove-error-when-tour-router-change branch from ad331e5 to a19bcd1 Compare May 23, 2024 09:51
@MitanOmar MitanOmar requested a review from c0rydoras May 23, 2024 09:58
@MitanOmar MitanOmar merged commit 74d8033 into adfinis:main May 23, 2024
9 checks passed
@MitanOmar MitanOmar deleted the remove-error-when-tour-router-change branch May 23, 2024 09:59
} catch (error) {
this.notify.error("unexpected error");
} catch {
/* eslint:disable:no-empty */
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
/* eslint:disable:no-empty */
// eslint-disable-line no-empty

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.

[BUG]: Tour yields two "unexpected error" notifications
3 participants