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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove unstable telemetry feature added for icons #2561
Conversation
7d6b1a4
to
f0be941
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On errant space but whatevs.
Not tophatted but LGTM. IIRC we're going to wait till after christmas and https://github.com/Shopify/web/pull/21923 is merged before we merge this
f0be941
to
e0d5385
Compare
The web PR needs to be merged first (https://github.com/Shopify/web/pull/21923) Then we can merge the core and polaris PRs whenever |
e0d5385
to
a17a3f2
Compare
WHY are these changes introduced?
We introduced these changes to gather some data in web about icons usage. We have finished gathering data, so there's no need to keep this code around any longer
WHAT is this pull request doing?
Reverts #2368
How to 馃帺
馃枼 Local development instructions
馃棐 General tophatting guidelines
馃搫 Changelog guidelines
Copy-paste this code in
playground/Playground.tsx
:馃帺 checklist
README.md
with documentation changes