Gleipnir/nidus-sync
Gleipnir/nidus-sync
7
0
Fork 0
Code Issues 54 Pull requests 3 Projects Releases Packages Wiki Activity Actions

Communication card: show address, reporter name, and public report ID #170

Open
ned wants to merge 1 commit from issue-165-address-reporter-list-view into main
pull from: issue-165-address-reporter-list-view
merge into: Gleipnir:main
Conversation 0 Commits 1 Files changed 4 +119 -29
ned commented 2026-07-24 16:06:54 +00:00
Member
Copy link

Adds address, reporter name, and public report ID to the communication list cards in the Communications Workbench.

Backend changes:

  • Added Address field to RelatedRecord struct, populated from report's AddressRaw
  • Added communicationReport struct (public_id, address)
  • Communication JSON response now includes a reports array with public_id and address for every report in the communication

Frontend changes:

  • Added CommunicationReport class and DTO
  • Communication cards now show:
    • Reporter name (from contact)
    • Address (or "N addresses" indicator when multiple distinct addresses exist)
    • Public report IDs (up to 2 shown inline, more shown as "+N more")

Closes #165

Adds address, reporter name, and public report ID to the communication list cards in the Communications Workbench. **Backend changes:** - Added `Address` field to `RelatedRecord` struct, populated from report's AddressRaw - Added `communicationReport` struct (`public_id`, `address`) - Communication JSON response now includes a `reports` array with public_id and address for every report in the communication **Frontend changes:** - Added `CommunicationReport` class and DTO - Communication cards now show: - Reporter name (from contact) - Address (or "N addresses" indicator when multiple distinct addresses exist) - Public report IDs (up to 2 shown inline, more shown as "+N more") Closes #165
Add address, reporter name, and public report ID to communication card
All checks were successful
/ golint (push) Successful in 20s
Details
/ pnpm-build (push) Successful in 1m8s
Details
4242445570 
The Communication list view cards now show three additional fields to help
admins reference specific reports during phone calls with the public:

- Reporter name (already available via comm.contact, now surfaced on the card)
- Address from the source/related reports, with a multiple-addresses indicator
  when a communication groups reports with different addresses
- Public report IDs (e.g. J8A3HD9RXGYY) for cross-referencing

Backend changes:
- Added Address field to platform.RelatedRecord, populated from each report's
  AddressRaw in CommunicationRelatedRecords
- Added communicationReport struct and Reports field to the communication JSON
  response, carrying public_id and address for every report in the communication
- Populated reports from the source report and all related records

Frontend changes:
- Added CommunicationReport class and CommunicationReportDTO interface
- Updated Communication.fromJSON to parse the new reports field
- ListCardCommunication.vue now shows reporter name, address (or multiple
  indicator), and public report IDs below the existing type icons + timestamp
eliribble force-pushed issue-165-address-reporter-list-view from 4242445570
All checks were successful
/ golint (push) Successful in 20s
Details
/ pnpm-build (push) Successful in 1m8s
Details
to 22fc442cee
All checks were successful
/ golint (push) Successful in 19s
Details
/ pnpm-build (push) Successful in 1m29s
Details
2026-08-29 22:18:16 +00:00 Compare
All checks were successful
/ golint (push) Successful in 19s
Details
/ pnpm-build (push) Successful in 1m29s
Details
This pull request has changes conflicting with the target branch.
  • resource/communication.go
  • ts/components/ListCardCommunication.vue
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin issue-165-address-reporter-list-view:issue-165-address-reporter-list-view
git switch issue-165-address-reporter-list-view

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff issue-165-address-reporter-list-view
git switch issue-165-address-reporter-list-view
git rebase main
git switch main
git merge --ff-only issue-165-address-reporter-list-view
git switch issue-165-address-reporter-list-view
git rebase main
git switch main
git merge --no-ff issue-165-address-reporter-list-view
git switch main
git merge --squash issue-165-address-reporter-list-view
git switch main
git merge --ff-only issue-165-address-reporter-list-view
git switch main
git merge issue-165-address-reporter-list-view
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
  
customer
Customers are aware of, and affected by the issue

  
prod
Issue has manifested in production

  
repro'd
Has been reproduced by engineering

  
retest
Issue is ready for retesting in staging

  
staging
Issue has manifested in staging

No labels
customer
prod
repro'd
retest
staging
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Gleipnir/nidus-sync!170
No description provided.