Skip to content

chore(cleanup): removing redundant rendering logic in telemetry pixel#26293

Merged
rusackas merged 1 commit into
masterfrom
cleanup-telemetry-component
Dec 18, 2023
Merged

chore(cleanup): removing redundant rendering logic in telemetry pixel#26293
rusackas merged 1 commit into
masterfrom
cleanup-telemetry-component

Conversation

@rusackas
Copy link
Copy Markdown
Member

SUMMARY

This duplicated logic/syntax slipped through on #26011 — I'm surprised eslint didn't find this redundancy, but whatever... it's gone now :)

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Copy link
Copy Markdown
Member

@michael-s-molina michael-s-molina left a comment

Choose a reason for hiding this comment

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

LGTM

@rusackas rusackas merged commit eb4b412 into master Dec 18, 2023
sadpandajoe added a commit to preset-io/superset that referenced this pull request Dec 22, 2023
sadpandajoe added a commit to preset-io/superset that referenced this pull request Jan 3, 2024
sadpandajoe added a commit to preset-io/superset that referenced this pull request Jan 13, 2024
@mistercrunch mistercrunch deleted the cleanup-telemetry-component branch March 26, 2024 18:06
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 4.0.0 First shipped in 4.0.0 labels Apr 17, 2024
vinothkumar66 pushed a commit to vinothkumar66/superset that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XS 🚢 4.0.0 First shipped in 4.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants