Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ai reply feedbacks #596

Merged
merged 26 commits into from
Jan 27, 2025
Merged

Ai reply feedbacks #596

merged 26 commits into from
Jan 27, 2025

Conversation

MrOrz
Copy link
Member

@MrOrz MrOrz commented Jan 26, 2025 •
edited
Loading

Collect anonymous feedback for AI replies

  • Install langfuse, add host and key
  • Implement button that can be voted by anonymous users
  • Apply babel and ES Module to langfuse SDK so that it can be built with NodeJS 16
upvote-downvote.mp4

@coveralls
Copy link

coveralls commented Jan 26, 2025 •
edited
Loading

Coverage Status

coverage: 71.733%. remained the same
when pulling 2f799dc on ai-reply-feedbacks
into 3ca739b on master.

MrOrz added 23 commits January 26, 2025 23:18
@MrOrz
chore(package.json): install Langfuse SDK
85d5931
@MrOrz
feat(AIReplySection): add voting buttons for AI replies
8b55537
@MrOrz
feat: Add vote handling placeholder for AI reply section
35d11c7
@MrOrz
feat: Implement Langfuse vote tracking for AI replies
aef3cdf
@MrOrz
style: Apply linter formatting to AIReplySection.js
3190e3c
@MrOrz
feat(AIReplySection): pass aiResponseId to handleVote and update Grap…
d91d8c4 
…hQL query for AI replies
@MrOrz
feat(.env): add Langfuse configuration to .env.sample
0e88460
@MrOrz
feat(next.config.js): add Langfuse SDK support with babel-loader conf…
f87e748 
…iguration
@MrOrz
feat(AIReplySection): integrate Langfuse SDK using next/config for co…
fdbcc4c 
…nfiguration
@MrOrz
style(AIReplySection): adjust styling for vote button and thumb icon …
8d0b42d 
…size
@MrOrz
refactor(AIReplySection): use Box prop instead
cdf047c
@MrOrz
refactor: Extract vote buttons into separate reusable component
b04f875
@MrOrz
style: Remove unused AIReplySection component
4108f5d
@MrOrz
feat(AIReplySection): add AIReplySection component with expandable vi…
f8217cb 
…ew and voting functionality
@MrOrz
feat: Add comment popover to VoteButtons with Langfuse feedback
87b04eb
@MrOrz
style: Format code with linter and improve readability
9ad0d8a
@MrOrz
fix(components): remove unused file file
183ac1c
@MrOrz
feat(VoteButtons): add voted style to highlight selected votes
3a8115b
@MrOrz
feat: Implement vote buttons with immediate feedback and optional com…
b0f301a 
…ments
@MrOrz
style: Apply linter formatting to VoteButtons component
39b7c42
@MrOrz
feat: Remove duplicate className props in VoteButtons component
13a8cb4
@MrOrz
feat(VoteButtons): improve vote handling and immediate feedback mecha…
cffe1de 
…nism
@MrOrz
feat(VoteButtons): refactor vote ID handling and improve type safety
b417176
@MrOrz MrOrz force-pushed the ai-reply-feedbacks branch from b8528bc to b417176 Compare January 26, 2025 15:18
@MrOrz MrOrz self-assigned this Jan 26, 2025
@MrOrz MrOrz merged commit dbec965 into master Jan 27, 2025
10 checks passed
@MrOrz MrOrz deleted the ai-reply-feedbacks branch January 27, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andyy0216 andyy0216 andyy0216 approved these changes

@bil4444 bil4444 bil4444 approved these changes

@nonumpa nonumpa Awaiting requested review from nonumpa

Assignees

@MrOrz MrOrz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MrOrz @coveralls @andyy0216 @bil4444