Skip to content

Dev#603

Merged
dbarzin merged 3 commits intomainfrom
dev
Apr 10, 2026
Merged

Dev#603
dbarzin merged 3 commits intomainfrom
dev

Conversation

@dbarzin
Copy link
Copy Markdown
Owner

@dbarzin dbarzin commented Apr 10, 2026

Summary by CodeRabbit

  • New Features

    • Added "Create Risk" button accessible to authorized users on the risk view page.
  • Style

    • Improved layout spacing for the risk name section and threshold color inputs.
    • Centered risk threshold preview badges for better visual alignment.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: d1c8ea66-26c6-4a10-af7a-3ee922563baf

📥 Commits

Reviewing files that changed from the base of the PR and between 4bc3b06 and 8fb8c05.

⛔ Files ignored due to path filters (26)
  • site/images/a1.fr.png is excluded by !**/*.png
  • site/images/a2.fr.png is excluded by !**/*.png
  • site/images/c1.fr.png is excluded by !**/*.png
  • site/images/c2.fr.png is excluded by !**/*.png
  • site/images/c3.fr.png is excluded by !**/*.png
  • site/images/c4.fr.png is excluded by !**/*.png
  • site/images/c5.fr.png is excluded by !**/*.png
  • site/images/config.fr.png is excluded by !**/*.png
  • site/images/d1.fr.png is excluded by !**/*.png
  • site/images/d2.fr.png is excluded by !**/*.png
  • site/images/d3.fr.png is excluded by !**/*.png
  • site/images/d4.fr.png is excluded by !**/*.png
  • site/images/domains.fr.png is excluded by !**/*.png
  • site/images/home1.fr.png is excluded by !**/*.png
  • site/images/home1.png is excluded by !**/*.png
  • site/images/home2.fr.png is excluded by !**/*.png
  • site/images/m1.fr.png is excluded by !**/*.png
  • site/images/m2.fr.png is excluded by !**/*.png
  • site/images/m3.fr.png is excluded by !**/*.png
  • site/images/plan.fr.png is excluded by !**/*.png
  • site/images/r1.fr.png is excluded by !**/*.png
  • site/images/r2.fr.png is excluded by !**/*.png
  • site/images/r3.fr.png is excluded by !**/*.png
  • site/images/search.fr.png is excluded by !**/*.png
  • site/images/tags.fr.png is excluded by !**/*.png
  • site/images/users.fr.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • resources/views/risks/scoring/form.blade.php
  • resources/views/risks/show.blade.php

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting


Walkthrough

Two Blade template files were updated with layout and styling adjustments. The first standardizes color input styling and adds centered alignment to a preview badge. The second expands the risk name cell width and conditionally adds a "New" button for authorized users.

Changes

Cohort / File(s) Summary
Risk Scoring Form Styling
resources/views/risks/scoring/form.blade.php
Standardized indentation of color input width styling (style="width:40px;") and added class="text-center" to the preview badge cell for improved alignment.
Risk Display Template Layout
resources/views/risks/show.blade.php
Increased risk name cell width from cell-lg-7 to cell-lg-10 and added conditional "New" button that displays for users with role 1 or 2, linking to /risk/create.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, hop, layout we mend,
Color boxes styled, aligned to the end,
A "New" button appears for the chosen few,
Grid cells resized—the form's looking new!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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.

@dbarzin dbarzin merged commit 1dc5540 into main Apr 10, 2026
1 check was pending
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.

1 participant