Skip to content

chore: dependencies upgrade - #48

Merged
simonhamp merged 3 commits into
NativePHP:mainfrom
SRWieZ:dependencies
Sep 18, 2024
Merged

chore: dependencies upgrade#48
simonhamp merged 3 commits into
NativePHP:mainfrom
SRWieZ:dependencies

Conversation

@SRWieZ

@SRWieZ SRWieZ commented Sep 16, 2024

Copy link
Copy Markdown
Member

No description provided.

@simonhamp

Copy link
Copy Markdown
Member

We use npm on the server... can you use npm locally too? This should help prevent conflicts on deployments (until we have a better way to handle this)

@SRWieZ

SRWieZ commented Sep 16, 2024

Copy link
Copy Markdown
Member Author

I'm using npm too, but then I saw a yarn.lock ^^
Should I delete yarn.lock then ?

@simonhamp

Copy link
Copy Markdown
Member

Hmmm npm is such a mess

I think this should be fine

@simonhamp
simonhamp merged commit e893f70 into NativePHP:main Sep 18, 2024
simonhamp added a commit that referenced this pull request Mar 30, 2026
… name

Implement HasName interface on User model with getFilamentName() that
falls back to display_name → name → email, preventing the ViewException
when the filament-impersonate banner renders for users with null names.

Closes #48

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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