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: loading tooltips before messages #7504

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

BeksOmega
Copy link
Collaborator

The basics

The details

Resolves

Fixes #7487
Fixes #4369

Proposed Changes

Removes references to runAfterPageLoad since these weren't working anyway.

Reason for Changes

Get rid of annoying console spam.

Test Coverage

Tested that tooltips for dropdowns and variables are still showing up correctly.

Documentation

N/A

Additional Information

N/A

@BeksOmega BeksOmega requested a review from a team as a code owner September 19, 2023 19:23
@github-actions github-actions bot added the PR: fix Fixes a bug label Sep 19, 2023
core/extensions.ts Outdated Show resolved Hide resolved
@BeksOmega BeksOmega enabled auto-merge (squash) September 20, 2023 14:04
@BeksOmega BeksOmega merged commit 12ac358 into google:develop Sep 20, 2023
7 checks passed
@BeksOmega BeksOmega deleted the fix/loading-tooltips branch May 14, 2024 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Playground receiving message warnings No Message String... warning in console
2 participants