Skip to content

Updated composer dependencies - #1262

Draft
korridor wants to merge 8 commits into
mainfrom
feature/composer-updates
Draft

korridor wants to merge 8 commits into
mainfrom
feature/composer-updates

Conversation

@korridor

@korridor korridor commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@korridor
korridor force-pushed the feature/composer-updates branch from b506bb1 to 47327aa Compare September 21, 2026 14:09
@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.73183% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.78%. Comparing base (c23f09f) to head (e1ec883).

Files with missing lines Patch % Lines
app/Service/Dto/ReportPropertiesDto.php 0.00% 7 Missing ⚠️
...pp/Http/Controllers/Api/V1/TimeEntryController.php 33.33% 4 Missing ⚠️
...RelationManagers/ProjectMembersRelationManager.php 57.14% 3 Missing ⚠️
...tionManagers/OwnedOrganizationsRelationManager.php 50.00% 3 Missing ⚠️
app/Http/Requests/V1/Report/ReportStoreRequest.php 66.66% 2 Missing ⚠️
...s/V1/TimeEntry/TimeEntryAggregateExportRequest.php 66.66% 2 Missing ⚠️
...uests/V1/TimeEntry/TimeEntryIndexExportRequest.php 66.66% 2 Missing ⚠️
app/Service/TimeEntryAggregationService.php 60.00% 2 Missing ⚠️
app/Service/DashboardService.php 80.00% 1 Missing ⚠️
app/Service/Import/ImportDatabaseHelper.php 50.00% 1 Missing ⚠️
... and 2 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1262      +/-   ##
============================================
+ Coverage     89.46%   92.78%   +3.32%     
- Complexity     2160     2167       +7     
============================================
  Files           305      305              
  Lines         10304    10314      +10     
============================================
+ Hits           9218     9570     +352     
+ Misses         1086      744     -342     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@korridor
korridor force-pushed the feature/composer-updates branch 2 times, most recently from cc51fc4 to 9b123af Compare September 22, 2026 10:12
Comment thread config/inertia.php Outdated
Comment thread public/fonts/filament/filament/inter/index.css Outdated
@Onatcer

Onatcer commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

app.blade.php:22 has a dead reference now, it was only there to exclude the filament css from the frontend but thats not necessary anymoer

@korridor korridor changed the title Updated compose dependencies Updated composer dependencies Sep 22, 2026
- Default INERTIA_SSR_ENABLED to false to match .env.example, .env.ci and
  .env.production, so existing setups keep the previous behaviour.
- Stop tracking the published Filament assets under /public/fonts/filament and
  add them to .gitignore, as recommended by the Filament docs. They are
  regenerated by filament:assets via the filament:upgrade post-autoload-dump
  hook. /public/css and /public/js are already ignored.
- Drop the now dead theme.css filter from the Vite asset list in app.blade.php,
  since the custom Filament theme was removed along with viteTheme().
@korridor
korridor force-pushed the feature/composer-updates branch from 1ab271b to f6d2556 Compare September 22, 2026 15:06
@korridor
korridor force-pushed the feature/composer-updates branch from f6d2556 to e1ec883 Compare September 22, 2026 16:01
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