Skip to content

refactor: consume FeedbackClient from @dyne/interfacer-client SDK - #880

Merged
phoebus-84 merged 3 commits into
mainfrom
feat/use-sdk-feedback-client
Jul 16, 2026
Merged

refactor: consume FeedbackClient from @dyne/interfacer-client SDK#880
phoebus-84 merged 3 commits into
mainfrom
feat/use-sdk-feedback-client

Conversation

@phoebus-84

Copy link
Copy Markdown
Collaborator

Summary

Replaces the raw REST + Zenroom signing code in lib/feedback.ts with the SDK's new client.feedback module.

Before

  • 231 lines of raw fetch + DID signing + Zenroom zencode_exec
  • Duplicated HTTP error handling and header construction

After

  • 95 lines — thin wrapper around client.feedback
  • Same API shape for backward compatibility
  • All signing + HTTP handled by the SDK

Depends on

SDK changes

  • New FeedbackClient class with 8 methods: createReview, getReviews, getReviewSummary, getUserReview, deleteReview, createComment, getComments, deleteComment
  • Integrated as client.feedback on the InterfacerClient facade
  • New feedbackUrl config field

@phoebus-84
phoebus-84 merged commit 26c3b1b into main Jul 16, 2026
1 check passed
@phoebus-84
phoebus-84 deleted the feat/use-sdk-feedback-client branch July 16, 2026 13:43
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.

1 participant