Skip to content

i18n#606

Merged
dbarzin merged 4 commits intomainfrom
dev
Apr 11, 2026
Merged

i18n#606
dbarzin merged 4 commits intomainfrom
dev

Conversation

@dbarzin
Copy link
Copy Markdown
Owner

@dbarzin dbarzin commented Apr 11, 2026

Summary by CodeRabbit

  • New Features

    • Enhanced risk scoring configuration with localized labels in German, English, and French.
    • Added exposure and vulnerability level defaults to scoring setup.
  • Improvements

    • Adjusted vulnerability section layout on scoring forms for better visibility.
  • Security

    • All application routes now require user authentication.

@dbarzin dbarzin merged commit cc39025 into main Apr 11, 2026
1 check was pending
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 11, 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: d2d897e9-90ec-4877-a40d-ad75d076299c

📥 Commits

Reviewing files that changed from the base of the PR and between 701134b and e92873a.

📒 Files selected for processing (6)
  • app/Http/Controllers/RiskScoringConfigController.php
  • resources/lang/de/cruds.php
  • resources/lang/en/cruds.php
  • resources/lang/fr/cruds.php
  • resources/views/risks/scoring/form.blade.php
  • routes/web.php

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting


Walkthrough

This pull request adds German localization for risk management features, introduces default label mappings in English and French translations, updates the RiskScoringConfigController to use translated defaults and improved admin authorization checks, adjusts the scoring form grid layout, and applies authentication middleware to web routes.

Changes

Cohort / File(s) Summary
Localization: German
resources/lang/de/cruds.php
Added comprehensive risk localization namespace with UI text for risk management pages, form fields, treatment statuses, risk levels, scoring configuration, color names, formula labels, and default threshold/level maps.
Localization: English & French
resources/lang/en/cruds.php, resources/lang/fr/cruds.php
Added new defaults section under risk_scoring containing label mappings for probability/exposure/vulnerability/impact levels and risk thresholds. Minor indentation adjustments in French file.
Risk Scoring Configuration
app/Http/Controllers/RiskScoringConfigController.php
Updated admin authorization to use isAdmin() method; replaced hardcoded French labels with localized translation strings in default config initialization; added exposure_levels and vulnerability_levels sections with translated labels; adjusted whitespace around commented code.
Scoring Form Layout
resources/views/risks/scoring/form.blade.php
Changed vulnerability section grid width from full width (cell-lg-12 cell-md-12) to half width (cell-lg-6 cell-md-6).
Web Routes Middleware
routes/web.php
Applied auth middleware to all routes within the main namespace group, requiring authentication for all listed endpoints.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Translations hop and dance in three tongues true,
Default labels bloom where risk metrics grew,
Auth guards the gates with a watchful stance,
While layouts reshape in a layout dance,
Risk management blooms in multilingual delight!

✨ 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.

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