-
Notifications
You must be signed in to change notification settings - Fork 24
Change third party logos to generic images #957
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
Conversation
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.
Pull request overview
This PR replaces third-party branded icon references with generic icon alternatives across the integrations documentation. The changes ensure consistent icon representation without relying on third-party logos or custom icon paths.
Changes:
- Replaced brand-specific icon references (e.g.,
simple-icons:snowflake,logos:vercel-icon) with generic alternatives - Removed custom icon path configuration for Rollbar in favor of a standard icon
- Applied consistent generic icons across similar integration types (analytics, code tools, insights)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 'Enhance error logging with DevCycle Feature and Variable data', | ||
| icon: '', | ||
| customIconPath: '/integrations/rollbar/rollbar-icon.svg' | ||
| icon: 'mdi:analytics', |
Copilot
AI
Feb 5, 2026
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.
The same 'mdi:analytics' icon is used for both Google Analytics (line 71) and Rollbar (line 79), despite these being different types of integrations (analytics vs error logging). Consider using a more appropriate icon for Rollbar, such as 'mdi:bug' or 'mdi:alert-circle' to better represent error logging functionality.
| icon: 'mdi:analytics', | |
| icon: 'mdi:bug', |
| description: 'Display code snippets for each variable used in a project.', | ||
| label: 'GitHub: Flag Code Usages', | ||
| icon: 'simple-icons:github', | ||
| icon: 'mdi:analytics', |
Copilot
AI
Feb 5, 2026
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.
The 'mdi:analytics' icon is reused for GitHub, Bitbucket, and GitLab 'Flag Code Usages' integrations. Since these integrations display code snippets rather than perform analytics, consider using a more semantically appropriate icon like 'mdi:code-tags' or 'mdi:file-code' to better represent code usage functionality.
| description: 'Display code snippets for each variable used in a project.', | ||
| label: 'Bitbucket: Flag Code Usages', | ||
| icon: 'simple-icons:bitbucket', | ||
| icon: 'mdi:analytics', |
Copilot
AI
Feb 5, 2026
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.
The 'mdi:analytics' icon is reused for GitHub, Bitbucket, and GitLab 'Flag Code Usages' integrations. Since these integrations display code snippets rather than perform analytics, consider using a more semantically appropriate icon like 'mdi:code-tags' or 'mdi:file-code' to better represent code usage functionality.
| description: 'Display code snippets for each variable used in a project.', | ||
| label: 'GitLab: Flag Code Usages', | ||
| icon: 'simple-icons:gitlab', | ||
| icon: 'mdi:analytics', |
Copilot
AI
Feb 5, 2026
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.
The 'mdi:analytics' icon is reused for GitHub, Bitbucket, and GitLab 'Flag Code Usages' integrations. Since these integrations display code snippets rather than perform analytics, consider using a more semantically appropriate icon like 'mdi:code-tags' or 'mdi:file-code' to better represent code usage functionality.
Deploying devcycle-docs with
|
| Latest commit: |
b00375f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e653cc33.devcycle-docs.pages.dev |
| Branch Preview URL: | https://update-integrations-third-pa.devcycle-docs.pages.dev |
Fix Spelling Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.