#405 - Laravel 11 upgrade

blumilksoftware / toby

HR software you love to hate

MIT License

17 stars 4 forks source link

#405 - Laravel 11 upgrade #413

Closed krzysztofrewak closed 7 months ago

krzysztofrewak commented 8 months ago

In this PR I have:

updated all dependencies including Laravel to new main version
reorganize app directories
reorganize Laravel structures

This should close #405.

krzysztofrewak commented 8 months ago

With last commit I removed rackbeat/laravel-ui-avatars. I replace it with really simple custom class.

mtracz commented 8 months ago

@EwelinaSkrzypacz If you run app in dev mode (npm run dev), open app via http not https. If you run app in prod mode (npm run build) https should works.

EwelinaSkrzypacz commented 8 months ago

On dev env I open app via http

On beta env I do nothing - just I made deploy

mtracz commented 8 months ago

@EwelinaSkrzypacz Looks like beta is working after configuring trusted proxies middleware.

EwelinaSkrzypacz commented 8 months ago

Thanks, beta is working right now - I can login and click through application.

But I have one problem - I can't create request (on local env and beta env) because I see 401 error in dev console. It's like request types don't fetch

sentry-io[bot] commented 7 months ago

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

‼️ Illuminate\Database\LazyLoadingViolationException: Attempted to lazy load [user] on model [Toby\Models\VacationRequest] but lazy loading is disabled. /app/Notifications/VacationRequestsSummaryNotif... View Issue
‼️ Illuminate\Database\LazyLoadingViolationException: Attempted to lazy load [user] on model [Toby\Models\VacationRequest] but lazy loading is disabled. /app/Slack/Elements/VacationRequestsAttachment.... View Issue

_{Did you find this useful? React with a 👍 or 👎}