Skip to content

Add Entity Relationship Diagrams#210

Merged
chao-xian merged 3 commits intomainfrom
add-model-diagrams
Jun 2, 2025
Merged

Add Entity Relationship Diagrams#210
chao-xian merged 3 commits intomainfrom
add-model-diagrams

Conversation

@chao-xian
Copy link
Copy Markdown
Contributor

@chao-xian chao-xian commented May 28, 2025

Adds our current domain model ERDs.

This is to help us to have something to refer to as we analyse and decide upon how to model supporting Welsh (and other) languages in our forms in the future.

We probably don't need one for Runner but I included one anyway for consistency.

These were generated by temporarily adding the main branch version of the Rails ERD gem (because there hasn't been a release for several years) and using the Mermaid generator option.

PR Checklist

  • If you are proposing a new decision record document, used the right template for that
    - (ADR, decision-record, engagement, research)
  • Set yourself as the Assignee
  • Tag anyone you would like to review, or @forms-design or @forms-devs
  • Fill in the template below

What

Describe what you have changed and why.

How to review

Describe the steps required to test the changes.

For example:

  1. Semantic: Do you agree with the changes?
  2. Syntactic: Spelling, grammar, etc.

Who can review

Describe who can review the changes. Or more importantly, list the people
that can't review, because they worked on it.

@chao-xian chao-xian force-pushed the add-model-diagrams branch from b2038d4 to 7d760ea Compare May 28, 2025 13:31
@chao-xian chao-xian force-pushed the add-model-diagrams branch from 08ddf5e to af2bbc2 Compare May 29, 2025 11:37
@chao-xian chao-xian marked this pull request as ready for review May 30, 2025 13:43
@chao-xian chao-xian merged commit 4830670 into main Jun 2, 2025
2 checks passed
@chao-xian chao-xian deleted the add-model-diagrams branch June 2, 2025 15:58
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