Skip to content

refactor: migrate review panel to permissions#1024

Draft
andyksaw wants to merge 2 commits intomainfrom
review-permissions
Draft

refactor: migrate review panel to permissions#1024
andyksaw wants to merge 2 commits intomainfrom
review-permissions

Conversation

@andyksaw
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Feb 28, 2026

Codecov Report

❌ Patch coverage is 19.35484% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.73%. Comparing base (7c71001) to head (3ca1718).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...Controllers/Manage/Activity/ActivityController.php 0.00% 6 Missing ⚠️
...ontrollers/Front/BanAppeal/BanAppealController.php 0.00% 4 Missing ⚠️
...ers/Review/BuilderRanks/BuilderRanksController.php 0.00% 4 Missing ⚠️
app/Domains/Permissions/AuthorizesPermissions.php 57.14% 3 Missing ⚠️
...rollers/Front/BuilderRankApplicationController.php 0.00% 3 Missing ⚠️
...trollers/Review/BanAppeals/BanAppealController.php 0.00% 3 Missing ⚠️
app/Http/Controllers/Manage/HomeController.php 0.00% 1 Missing ⚠️
app/Http/Controllers/Review/HomeController.php 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1024      +/-   ##
============================================
+ Coverage     40.69%   41.73%   +1.03%     
+ Complexity     1175     1132      -43     
============================================
  Files           312      307       -5     
  Lines          5472     5363     -109     
============================================
+ Hits           2227     2238      +11     
+ Misses         3245     3125     -120     
Files with missing lines Coverage Δ
app/AppServiceProvider.php 94.28% <ø> (-0.84%) ⬇️
...Domains/Permissions/PermissionsServiceProvider.php 70.37% <100.00%> (+2.37%) ⬆️
app/Http/Controllers/Manage/HomeController.php 0.00% <0.00%> (ø)
app/Http/Controllers/Review/HomeController.php 0.00% <0.00%> (ø)
app/Domains/Permissions/AuthorizesPermissions.php 50.00% <57.14%> (+13.63%) ⬆️
...rollers/Front/BuilderRankApplicationController.php 0.00% <0.00%> (ø)
...trollers/Review/BanAppeals/BanAppealController.php 0.00% <0.00%> (ø)
...ontrollers/Front/BanAppeal/BanAppealController.php 0.00% <0.00%> (ø)
...ers/Review/BuilderRanks/BuilderRanksController.php 0.00% <0.00%> (ø)
...Controllers/Manage/Activity/ActivityController.php 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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