Pr 3266 fix #3322
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| --- | ||
| "@trigger.dev/core": patch | ||
| "@trigger.dev/sdk": patch | ||
| --- | ||
|
|
||
| feat: Add OpenClaw agent integration with Slack webhooks | ||
|
|
||
| Implements Phase 1 MVP for AI agent platform allowing users to create agents through setup form (/agents/setup). Agents are stored in database with configuration (model, platform, tools). Slack webhook receives messages and triggers agent responses. Includes agent management UI and webhook integration infrastructure. |
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,32 +1,20 @@ | ||
| export function LogoIcon({ className }: { className?: string }) { | ||
| return ( | ||
| <svg | ||
| width="32" | ||
| height="32" | ||
| viewBox="0 0 32 32" | ||
| fill="none" | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| className={className} | ||
| > | ||
| <rect width="32" height="32" rx="8" fill="#2E7D32" /> | ||
| <path | ||
| d="M16 6L24 24H8L16 6Z" | ||
| fill="white" | ||
| fillOpacity="0.9" | ||
| /> | ||
| <circle cx="16" cy="20" r="2" fill="#2E7D32" /> | ||
| </svg> | ||
| ); | ||
| } | ||
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
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,190 +1,32 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| export function LogoType({ className }: { className?: string }) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| return ( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <svg | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| viewBox="0 0 520 80" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| xmlns="http://www.w3.org/2000/svg" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| className={className} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| > | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <defs> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <linearGradient id="logoGradient" x1="0%" y1="0%" x2="100%" y2="0%"> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <stop offset="0%" stopColor="#2E7D32" /> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <stop offset="100%" stopColor="#43A047" /> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| </linearGradient> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| </defs> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| {/* Icon */} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <rect x="0" y="8" width="64" height="64" rx="14" fill="url(#logoGradient)" /> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <path d="M32 20L46 56H18L32 20Z" fill="white" fillOpacity="0.9" /> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+9
to
+16
Contributor
There was a problem hiding this comment. 🧩 Analysis chain🏁 Script executed: # Find and read the LogoType.tsx file
fd -t f "LogoType.tsx" apps/Repository: triggerdotdev/trigger.dev Length of output: 109 🏁 Script executed: # Search for other uses of "logoGradient" or similar gradient patterns in the codebase
rg "logoGradient|linearGradient.*id=" --type tsx --type tsRepository: triggerdotdev/trigger.dev Length of output: 97 🏁 Script executed: # Check the current implementation of LogoType.tsx
cat apps/webapp/app/components/LogoType.tsxRepository: triggerdotdev/trigger.dev Length of output: 1022 🏁 Script executed: # Find components that use LogoType to understand if multiple instances appear on same page
rg "LogoType" --type tsx --type ts -B 2 -A 2Repository: triggerdotdev/trigger.dev Length of output: 97 🏁 Script executed: # Search for other gradient patterns and inline SVG IDs (broader search)
rg "id=\"[^\"]*Gradient|url\(#" apps/webapp --type-listRepository: triggerdotdev/trigger.dev Length of output: 459 🏁 Script executed: # Search for logoGradient references
rg "logoGradient" apps/Repository: triggerdotdev/trigger.dev Length of output: 310 🏁 Script executed: # Find imports of LogoType to understand usage patterns
rg "import.*LogoType|from.*LogoType" apps/Repository: triggerdotdev/trigger.dev Length of output: 155 🏁 Script executed: # Check if LoginPageLayout mentioned in review actually uses LogoType
rg "LogoType" apps/webapp/app/components/LoginPageLayout.tsxRepository: triggerdotdev/trigger.dev Length of output: 152 🏁 Script executed: # Find all imports of LogoType across the entire codebase
rg "import.*LogoType|from.*LogoType" apps/Repository: triggerdotdev/trigger.dev Length of output: 155 🏁 Script executed: # Search for other inline SVG patterns with IDs to assess collision risk
rg "id\s*=\s*['\"]" apps/webapp --max-count=50 | head -40Repository: triggerdotdev/trigger.dev Length of output: 6339 🏁 Script executed: # Check if there are other components in same directory that might have SVG elements
ls -la apps/webapp/app/components/ | grep -E "\.(tsx?|jsx?)$" | head -20Repository: triggerdotdev/trigger.dev Length of output: 1470 🏁 Script executed: # Search for any other gradient definitions or SVG markers/patterns
rg "linearGradient|pattern|marker" apps/webapp/app --max-count=20Repository: triggerdotdev/trigger.dev Length of output: 50383 🏁 Script executed: # Check if LogoType is used anywhere else besides LoginPageLayout
rg "LogoType" apps/ -A 3 -B 1 2>/dev/null | head -100Repository: triggerdotdev/trigger.dev Length of output: 1234 🏁 Script executed: # Look at LoginPageLayout to see how LogoType is used
cat apps/webapp/app/components/LoginPageLayout.tsxRepository: triggerdotdev/trigger.dev Length of output: 3938 🏁 Script executed: # Search for any scenarios where the page might render multiple SVGs with inline gradients
rg "linearGradient|<svg" apps/webapp/app/components/LoginPageLayout.tsxRepository: triggerdotdev/trigger.dev Length of output: 51 Use The fixed Proposed fix+import { useId } from "react";
+
export function LogoType({ className }: { className?: string }) {
+ const gradientId = useId();
return (
<svg
viewBox="0 0 520 80"
xmlns="http://www.w3.org/2000/svg"
className={className}
>
<defs>
- <linearGradient id="logoGradient" x1="0%" y1="0%" x2="100%" y2="0%">
+ <linearGradient id={gradientId} x1="0%" y1="0%" x2="100%" y2="0%">
<stop offset="0%" stopColor="`#2E7D32`" />
<stop offset="100%" stopColor="`#43A047`" />
</linearGradient>
</defs>
{/* Icon */}
- <rect x="0" y="8" width="64" height="64" rx="14" fill="url(`#logoGradient`)" />
+ <rect x="0" y="8" width="64" height="64" rx="14" fill={`url(#${gradientId})`} />📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <circle cx="32" cy="48" r="3.5" fill="#2E7D32" /> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| {/* Text */} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| <text | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| x="80" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| y="55" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fontFamily="Geist Variable, Inter, Helvetica Neue, sans-serif" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fontSize="42" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fontWeight="600" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| letterSpacing="-0.02em" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fill="currentColor" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| > | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Trigger.dev | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| </text> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| </svg> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
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.
🚩 LogoIcon and LogoType completely replaced with different brand identity
The Trigger.dev logo SVGs in
LogoIcon.tsxandLogoType.tsxhave been entirely replaced. The original logo was a detailed geometric shape with a green-to-yellow gradient (#41FF54to#E7FF52), consistent with Trigger.dev's established brand. The new logo is a green rounded rectangle (#2E7D32) with a simple white triangle and dot — a completely different design language. The LogoType also replaces the detailed path-based wordmark with an SVG<text>element rendering "Trigger.dev", which may render differently across browsers depending on font availability (Geist Variable, Inter, Helvetica Neue, sans-serif). This is a significant brand change that should be verified as intentional.Was this helpful? React with 👍 or 👎 to provide feedback.