Conversation
- Fix GraphQL enum case sensitivity issue - Convert lowercase level values to uppercase - Match backend ProjectLevel enum expectations - Resolves project level filtering bug
Resolves OWASP#3599 - Update grid layout classes to use mobile-first approach - Add responsive breakpoints: sm:, md:, lg:, xl: - Mobile: 1 column, Tablet: 2 columns, Desktop: 2-3 columns - Improve spacing with responsive gap values - Add col-span-full for proper empty state layout - Prevent overflow and layout breaking on mobile devices Result: ✅ Fully responsive layout across all screen sizes ✅ No overflow or layout breaking on mobile devices ✅ Improved user experience on smaller screens
|
PR validation failed: No linked issue and no valid closing issue reference in PR description |
|
Caution Review failedThe pull request is closed. Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings. WalkthroughThe metrics dashboard page component was updated with three main changes: project-level filter values were converted to uppercase string literals, the metrics grid layout was restructured to be responsive with defined breakpoints across different screen sizes, and the "No metrics found" message container was styled to span the full grid column. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
|
1 participant
STOP AND READ BEFORE SUBMITTING! REMOVE THIS PARAGRAPH BEFORE OPENING THE PR
Thank you for your interest in contributing to OWASP Nest!
Before starting any work, all external contributors must first be assigned to an issue in the repository.
This is a mandatory step in the OWASP Nest workflow and ensures that effort is coordinated, approved, and tracked properly.
Exception: OWASP leaders are not required to follow this rule —- just make sure your username is included in the exception list.
If you were not assigned to the issue you are trying to resolve, stop right now and do NOT create this PR.
Unassigned pull requests are automatically closed by our workflows — please don't waste your time.
If you want to be assigned on any available issue, comment on it and wait for confirmation from the maintainers or project leads.
Proposed change
Resolves #(put the issue number here)
Add the PR description here.
Checklist
make check-testlocally: all warnings addressed, tests passed