-
Notifications
You must be signed in to change notification settings - Fork 2
updated wording to match free plan / verified HIW component #158
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -27,15 +27,15 @@ export function HowItWorks() { | |
| <TabsList className="grid w-full grid-cols-2 max-w-md mx-auto mb-8 bg-jules-darker border-jules-primary"> | ||
| <TabsTrigger | ||
| value="hosted" | ||
| className="data-[state=active]:bg-jules-primary text-white cursor-pointer" | ||
| className="data-[state=active]:bg-jules-primary text-white cursor-pointer font-semibold" | ||
| > | ||
| Hosted | ||
| Hosted <span className="text-xs font-normal">(1min)</span> | ||
| </TabsTrigger> | ||
| <TabsTrigger | ||
| value="self-hosted" | ||
| className="data-[state=active]:bg-jules-primary text-white cursor-pointer" | ||
| className="data-[state=active]:bg-jules-primary text-white cursor-pointer font-semibold" | ||
| > | ||
| Self-Hosted | ||
| Self-Hosted <span className="text-xs font-normal">(15min)</span> | ||
| </TabsTrigger> | ||
| </TabsList> | ||
|
|
||
|
|
@@ -98,7 +98,7 @@ export function HowItWorks() { | |
| </Card> | ||
| </div> | ||
|
|
||
| <div className="text-center mt-8"> | ||
| <div className="text-center mt-8 invisible"> | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using the <div className="text-center mt-8 hidden"> |
||
| <Link | ||
| href="https://github.com/ihildy/jules-task-queue/blob/main/HOSTED.md" | ||
| className="inline-flex items-center text-jules-cyan hover:text-jules-cyan/80 transition-colors" | ||
|
|
@@ -127,8 +127,8 @@ export function HowItWorks() { | |
| </code> | ||
| </div> | ||
| <CardDescription className="text-gray-300"> | ||
| Set up GitHub token, webhook secret, database URL, and cron | ||
| secret | ||
| Create a GitHub app, setup a webhook secret, database URL, | ||
| and cron secret | ||
| </CardDescription> | ||
| </CardContent> | ||
| </Card> | ||
|
|
@@ -159,16 +159,16 @@ export function HowItWorks() { | |
| 3 | ||
| </div> | ||
| <CardTitle className="text-xl text-white mb-4"> | ||
| Configure GitHub Webhooks | ||
| Watch It Work | ||
| </CardTitle> | ||
| <div className="rounded-lg p-4 mb-4 bg-jules-dark"> | ||
| <code className="text-sm text-jules-accent"> | ||
| Repository → Settings → Webhooks | ||
| <code className="text-sm text-jules-pink"> | ||
| Auto-retry every 30 minutes | ||
| </code> | ||
| </div> | ||
| <CardDescription className="text-gray-300"> | ||
| Add webhook URL pointing to your domain and enable | ||
| “Issues” events | ||
| Tasks that hit the limit are automatically queued and | ||
| retried every 30 minutes | ||
| </CardDescription> | ||
| </CardContent> | ||
| </Card> | ||
|
|
||
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.
Using
<br />tags for spacing is generally discouraged in favor of CSS for layout and styling. For better semantic structure and maintainability, you could wrap the new disclaimer text in a<p>tag and use a margin utility for spacing. Since the parentAlertDescriptioncomponent renders as adiv, this is semantically correct and will be styled appropriately.