Skip to content

feat: update report a bug text in the feedback panel#319

Merged
apinkert merged 3 commits into
RedHatInsights:masterfrom
apinkert:update-feedback
May 13, 2026
Merged

feat: update report a bug text in the feedback panel#319
apinkert merged 3 commits into
RedHatInsights:masterfrom
apinkert:update-feedback

Conversation

@apinkert
Copy link
Copy Markdown
Collaborator

@apinkert apinkert commented May 13, 2026

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: d94f3990-c7f7-4bc7-a15b-a1aaeae6504d

📥 Commits

Reviewing files that changed from the base of the PR and between d6983fa and 36e57ef.

📒 Files selected for processing (1)
  • src/components/HelpPanel/HelpPanelTabs/Feedback/FeedbackPanel.tsx

Summary by CodeRabbit

  • New Features
    • The Help panel's feedback section now provides a direct support-case link in the "report bug" card, streamlining issue reporting and support access.
    • Clicking the support-case link no longer triggers the card's navigation, so the link opens without activating the surrounding card.

Walkthrough

The PR updates FeedbackPanel to compute an isReportBug flag per card and render the "report bug" card body with the localized config.body plus an "open support case" link (which stops event propagation); other cards keep the prior localized body rendering.

Changes

Bug Report Card with Support Link

Layer / File(s) Summary
Conditional card body rendering
src/components/HelpPanel/HelpPanelTabs/Feedback/FeedbackPanel.tsx
Adds isReportBug (config.page === 'reportBugOne') and replaces the uniform CardBody with a conditional: for the report-bug card render config.body followed by an "open support case" link whose onClick/onKeyDown stop propagation; otherwise render intl.formatMessage(config.body) as before.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the issue reference and a screenshot, but is missing the required template sections like detailed summary, accessibility checklist, and verification steps. Complete the description template by adding a 2-3 sentence summary of changes, before/after context, accessibility notes, and the full checklist.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: updating the 'report a bug' text in the feedback panel component.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/components/HelpPanel/HelpPanelTabs/Feedback/FeedbackPanel.tsx`:
- Around line 186-203: The support-case anchor inside FeedbackPanel's CardBody
triggers the parent card's onClick; update the Content anchor (the element
rendering href SUPPORT_CASE_URL inside the isReportBug branch) to stop
propagation and avoid activating the card when the link is used—add an onClick
handler that calls event.stopPropagation() (and also handle onKeyDown to stop
propagation for Enter/Space for accessibility) so the link opens normally
without triggering the card's onClick.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 736db59f-dc03-4eb1-bb6c-2512bdbca1c4

📥 Commits

Reviewing files that changed from the base of the PR and between ea808f3 and d6983fa.

📒 Files selected for processing (1)
  • src/components/HelpPanel/HelpPanelTabs/Feedback/FeedbackPanel.tsx

Comment thread src/components/HelpPanel/HelpPanelTabs/Feedback/FeedbackPanel.tsx
@apinkert apinkert merged commit d38310c into RedHatInsights:master May 13, 2026
9 checks passed
@apinkert apinkert deleted the update-feedback branch May 13, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants