Skip to content

Refactor parents as contacts#291

Merged
paulrobertlloyd merged 8 commits into
mainfrom
refactor-parents
May 14, 2026
Merged

Refactor parents as contacts#291
paulrobertlloyd merged 8 commits into
mainfrom
refactor-parents

Conversation

@paulrobertlloyd
Copy link
Copy Markdown
Collaborator

@paulrobertlloyd paulrobertlloyd commented May 13, 2026

Contact (formerly parent) details are now storied in global context, meaning editing a contact in one location will update it in another. This means contacts behave more like they do in production, and will be useful as we think about how contacts should look and behave going forward.

It also means contacts are structured in much the same way as other entities in the prototype.

@paulrobertlloyd paulrobertlloyd requested a review from malross May 13, 2026 17:32
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 13, 2026 17:34 Inactive
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 13, 2026 17:45 Inactive
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 13, 2026 17:46 Inactive
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 14, 2026 09:28 Inactive
Comment thread app/models/clinic-appointment.js
Comment thread app/controllers/reply.js Outdated
Copy link
Copy Markdown
Collaborator

@malross malross left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one request for change and even then, it's just in a comment. As someone new to the codebase, it's the kind of comment that I would've valued more of.

Comment thread app/models/reply.js Outdated
Comment thread app/models/reply.js
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 14, 2026 14:42 Inactive
@paulrobertlloyd paulrobertlloyd temporarily deployed to mavis-protot-refactor-p-s51vnh May 14, 2026 15:11 Inactive
@paulrobertlloyd paulrobertlloyd requested a review from malross May 14, 2026 15:11
@paulrobertlloyd paulrobertlloyd merged commit 6df3389 into main May 14, 2026
1 check passed
@paulrobertlloyd paulrobertlloyd deleted the refactor-parents branch May 14, 2026 15:20
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.

2 participants