Skip to content

feat: show user facing exception for message generation in ChattyLLM#608

Merged
lukasdotcom merged 1 commit into
mainfrom
user-facing-exception
Jul 21, 2026
Merged

feat: show user facing exception for message generation in ChattyLLM#608
lukasdotcom merged 1 commit into
mainfrom
user-facing-exception

Conversation

@lukasdotcom

Copy link
Copy Markdown
Member

Pretty helpful for multimodal so the user knows that a certain file type doesn't work.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@lukasdotcom
lukasdotcom requested a review from janepie July 16, 2026 12:55

@julien-nc julien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can fix the psalm issue by changing line 107 of lib/Migration/Version030500Date20260715083046.php from

$userIds = array_unique(array_merge($userIdsChat, $userIdsAssignments));

to

$userIds = array_values(array_unique(array_map('strval', array_merge($userIdsChat, $userIdsAssignments))));

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
@lukasdotcom
lukasdotcom force-pushed the user-facing-exception branch from 4003bcb to 64f7c22 Compare July 21, 2026 12:20
@lukasdotcom
lukasdotcom merged commit 135c975 into main Jul 21, 2026
14 checks passed
@lukasdotcom
lukasdotcom deleted the user-facing-exception branch July 21, 2026 12:22
@lukasdotcom

Copy link
Copy Markdown
Member Author

You can fix the psalm issue by changing line 107 of lib/Migration/Version030500Date20260715083046.php from

$userIds = array_unique(array_merge($userIdsChat, $userIdsAssignments));

to

$userIds = array_values(array_unique(array_map('strval', array_merge($userIdsChat, $userIdsAssignments))));

A rebase can also fix it thanks to your fix in main.

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