Skip to content

ADFA-4294 feat(feedback): pure domain slice for operator feedback (em…#95

Merged
luisguzman-adfa merged 1 commit into
mainfrom
feat/ADFA-4294-feedback-domain
Jun 29, 2026
Merged

ADFA-4294 feat(feedback): pure domain slice for operator feedback (em…#95
luisguzman-adfa merged 1 commit into
mainfrom
feat/ADFA-4294-feedback-domain

Conversation

@luisguzman-adfa

Copy link
Copy Markdown
Collaborator

…ail-first)

Transport-agnostic foundation for the feedback feature (no UI yet):

  • FeedbackType, RootfsChecksumStatus: stable English labels for subject/body/JSON.
  • FeedbackPayload: immutable value object (+ builder) with the agreed schema.
  • FeedbackRenderer -> EmailContent: renders the email-first transport with the shared-inbox filtering hooks (recipient feedback+k2go@ via subaddressing, [K2Go] subject prefix, 'Product: K2Go' body); optional fields omitted when absent.
  • JUnit tests for subject/body rendering, optional fields and stack-trace section.

Same fields map 1:1 to the future Cloudflare Worker JSON, so adding that transport is additive. Pure Java (no Android/I/O), unit-testable.

…ail-first)

Transport-agnostic foundation for the feedback feature (no UI yet):
- FeedbackType, RootfsChecksumStatus: stable English labels for subject/body/JSON.
- FeedbackPayload: immutable value object (+ builder) with the agreed schema.
- FeedbackRenderer -> EmailContent: renders the email-first transport with the
  shared-inbox filtering hooks (recipient feedback+k2go@ via subaddressing,
  [K2Go] subject prefix, 'Product: K2Go' body); optional fields omitted when absent.
- JUnit tests for subject/body rendering, optional fields and stack-trace section.

Same fields map 1:1 to the future Cloudflare Worker JSON, so adding that transport
is additive. Pure Java (no Android/I/O), unit-testable.
@luisguzman-adfa luisguzman-adfa merged commit a5101a8 into main Jun 29, 2026
1 check passed
@luisguzman-adfa luisguzman-adfa deleted the feat/ADFA-4294-feedback-domain branch June 29, 2026 21:23
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