Skip to content

fix: dim labels and reposition note for webhook-driven readonly fields#148

Open
R-Hart80 wants to merge 1 commit into
CatholicOS:devfrom
R-Hart80:fix/webhook-driven-readonly-fields
Open

fix: dim labels and reposition note for webhook-driven readonly fields#148
R-Hart80 wants to merge 1 commit into
CatholicOS:devfrom
R-Hart80:fix/webhook-driven-readonly-fields

Conversation

@R-Hart80
Copy link
Copy Markdown
Contributor

Closes #5

Problem

When isWebhookDriven is true, the four GitHub-integration input fields (repo_owner, repo_name, branch, git_ontology_path) were already rendered with readOnly and cursor-not-allowed opacity-60 on the inputs themselves. However, their labels remained at full opacity, which was visually inconsistent — the fields looked disabled but the labels did not.

Additionally, the explanatory note "Repository fields are managed by the GitHub integration." was placed between the first two-column grid (repo owner + name) and the second grid (branch + file path), making it ambiguous whether it applied to all four fields or only the bottom two.

Changes

  • app/projects/[id]/settings/page.tsx
    • Added isWebhookDriven && "opacity-60" to the className of all four field labels so they dim in sync with their inputs.
    • Moved the explanatory note to appear after both field grids so it clearly covers all four read-only fields as a group.

Test plan

  • Open a project that has a GitHub webhook configured (isWebhookDriven === true).
  • Navigate to Settings → GitHub Integration.
  • Verify the four labels ("Repository owner", "Repository name", "Branch", "File path") are visually dimmed (opacity-60) alongside their inputs.
  • Verify the note "Repository fields are managed by the GitHub integration." appears below all four fields, not between them.
  • Open a project without GitHub integration — verify labels are full opacity and inputs are editable.

🤖 Generated with Claude Code

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 13, 2026

Warning

Rate limit exceeded

@R-Hart80 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 44 minutes and 33 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6a5a1c2d-2c08-49e4-a1da-0636d9938d76

📥 Commits

Reviewing files that changed from the base of the PR and between 0233a83 and 1e51ffb.

📒 Files selected for processing (1)
  • app/projects/[id]/settings/page.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@JohnRDOrazio JohnRDOrazio added the bug Something isn't working label Apr 13, 2026
@damienriehl damienriehl force-pushed the fix/webhook-driven-readonly-fields branch from 207119a to c9d1614 Compare May 2, 2026 13:13
@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

When a project is webhook-driven, the four GitHub integration input
fields (repo owner, repo name, branch, file path) were already marked
readOnly and visually styled as disabled on the inputs themselves.
This commit extends the visual treatment to their labels by applying
opacity-60 when isWebhookDriven is true, and moves the explanatory
note ("Repository fields are managed by the GitHub integration.")
to appear after all four fields so it clearly covers the whole group.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@R-Hart80 R-Hart80 force-pushed the fix/webhook-driven-readonly-fields branch from c9d1614 to 1e51ffb Compare May 11, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify: read-only upstream sync fields when webhook-driven

2 participants