Add logos to all integrations on overview and individual pages #494
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-Authored-By: Ryan Seams <[email protected]>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor
|
Please remove the Remove this. We don't need an image of the logo on the page itself |
Per PR feedback from m-ods, removed <img> tags from all 14 individual integration pages. Logos are now only displayed on the integrations overview page (index.mdx), not on the individual integration pages. Co-Authored-By: Ryan Seams <[email protected]>
Contributor
Author
Restore missing spaces after periods in four integration pages: - haystack.mdx: applications.The -> applications. The - postman.mdx: endpoints.The -> endpoints. The - telnyx.mdx: services.By -> services. By - power-automate.mdx: services.[Azure -> services. [Azure These spacing issues were accidentally introduced when adding logos. Co-Authored-By: Ryan Seams <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add logos to all integrations on overview and individual pages
Summary
Added PNG logos with transparent backgrounds to all integrations that were missing them. Previously, only Livekit, Pipecat, and Vapi had logos on the integrations overview page. This PR adds logos for:
No-code tools: Zapier, Make, Postman
Meeting transcriber tools: Recall.ai, Zoom RTMS
Telephony tools: Telnyx, Twilio, Amazon Connect, Genesys Cloud
Community-maintained tools: LangChain, Power Automate, Semantic Kernel, Activepieces, Haystack, Cloudflare, Relay.app, Bubble, Pipedream
Changes include:
fern/assets/img/integrations/fern/pages/06-integrations/index.mdxto replace Font Awesome icons with image logos for all integrationsReview & Testing Checklist for Human
CRITICAL ISSUE - MUST FIX BEFORE MERGE:
langchain.png- This file is an HTML page (429KB) instead of an actual PNG image. Download the correct LangChain logo from their official brand assets or use a different source. The page will be broken until this is fixed.High Priority:
fern docs devlocally to verify all logos display correctly with proper transparency and sizingpostman.mdxline 9,power-automate.mdxline 9,telnyx.mdxline 9,haystack.mdxline 9)Medium Priority:
width="200"but overview page cards don't specify width. Verify sizing looks good across different screen sizes../../assets/img/integrations/work correctly in the Fern build systemTest Plan
fern docs devto start local documentation server/docs/integrationsand verify all integration cards show logos instead of Font Awesome iconsNotes
openapi.ymlandasyncapi.ymlare unrelated to these changesSession: https://app.devin.ai/sessions/fd62370fa6a04326a290ebdefa807b06
Requested by: Ryan Seams ([email protected])