Skip to content

feat: ai hint in chat - #329

Merged
lukasdotcom merged 3 commits into
mainfrom
feat/ai-hint-chat
Aug 7, 2025
Merged

feat: ai hint in chat#329
lukasdotcom merged 3 commits into
mainfrom
feat/ai-hint-chat

Conversation

@lukasdotcom

@lukasdotcomlukasdotcom commented Aug 7, 2025•
edited
Loading

Copy link
Copy Markdown
Member
imageResolves: #316

@lukasdotcom
feat: ai hint in chat
7222e34
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
@janepie

janepie commented Aug 7, 2025

Copy link
Copy Markdown
Member

Looks nice! I would maybe adjust the wording here as this hint is also shown when there is no output yet, and it's not possible to adjust anything. Maybe "Output shown here is generated by AI. Make sure to always double-check."
Wdyt @marcoambrosini ?

@lukasdotcom
better wording
6385102
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
@lukasdotcom

lukasdotcom commented Aug 7, 2025•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

Looks nice! I would maybe adjust the wording here as this hint is also shown when there is no output yet, and it's not possible to adjust anything. Maybe "Output shown here is generated by AI. Make sure to always double-check." Wdyt @marcoambrosini ?

I like your wording more and pushed a commit and updated the screenshot based on it. For reference the old wording was: "This output was generated by AI. Make sure to double-check and adjust."

julien-nc
julien-nc requested changes Aug 7, 2025
View reviewed changes
Comment threadsrc/components/ChattyLLM/ChattyLLMInputForm.vue Outdated
janepie
janepie approved these changes Aug 7, 2025
View reviewed changes

@janepiejanepie 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.

I would still leave out the "adjust", but I can accept it :D

julien-nc
julien-nc approved these changes Aug 7, 2025
View reviewed changes

@julien-ncjulien-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.

I would leave out the "and adjust" too.

@lukasdotcom
translate it
da039fe
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
@lukasdotcom

lukasdotcom commented Aug 7, 2025

Copy link
Copy Markdown
MemberAuthor

I removed the adjust part.

@lukasdotcom
lukasdotcom merged commit 51ec0b7 into mainAug 7, 2025
9 checks passed
@lukasdotcom
lukasdotcom deleted the feat/ai-hint-chat branch August 7, 2025 15:21
@janepiejanepie mentioned this pull request Aug 7, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julien-ncjulien-ncjulien-nc approved these changes
@janepiejanepiejanepie approved these changes
@marcoambrosinimarcoambrosiniAwaiting requested review from marcoambrosini

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AI hint in Chat UI

3 participants

@lukasdotcom@janepie@julien-nc