-
Notifications
You must be signed in to change notification settings - Fork 154
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
Add icons still missing after recent update #34
Comments
I see a PowerBI Embedded but no Native PowerBI is this by design? Also PurView will be handy |
@asos-eugeneniemand Purview icons will be added as part of #37 |
Agree that it would be helpful to have an icon for (regular) Power BI. I could add it to #37 and use the same icon provided for Embedded. |
@travisnielsen If you could, that would be awesome! 👍 |
Power BI icon added to the Analytics section. I also moved Power BI Embedded from the Databases section to Analytics. |
The recent update from #31 update (almost) all icons. Besides from some image corruption (being addressed in #32), it needs to be checked that none of the requested icon are missing.
From the overview of #29, the following icon might be missing:
Also, the currently open MRs could also contain icons that are not in the current set:
Both need to be verified, and either marked as complete or have a linked issue or MR that adds them.
This issue is a follow-up to #29 and related MR(s).
The text was updated successfully, but these errors were encountered: