Skip to content

Remove Relationship caches#670

Merged
jkoenig134 merged 15 commits into
release/v7from
remove-relationship-caches
Jul 18, 2025
Merged

Remove Relationship caches#670
jkoenig134 merged 15 commits into
release/v7from
remove-relationship-caches

Conversation

@jkoenig134
Copy link
Copy Markdown
Contributor

@jkoenig134 jkoenig134 commented Jul 18, 2025 •
edited
Loading

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.
  • I self-reviewed the PR.

@jkoenig134 jkoenig134 added the breaking-change A breaking change label Jul 18, 2025
@jkoenig134 jkoenig134 marked this pull request as ready for review July 18, 2025 09:43
@britsta britsta changed the title Remove relationship caches Remove Relationship caches Jul 18, 2025
britsta
britsta requested changes Jul 18, 2025
View reviewed changes
Comment thread packages/transport/src/modules/relationships/RelationshipsController.ts Outdated
Comment thread packages/transport/src/modules/relationships/RelationshipsController.ts Outdated
@jkoenig134 jkoenig134 requested a review from britsta July 18, 2025 10:03
tnotheis
tnotheis previously approved these changes Jul 18, 2025
View reviewed changes
britsta
britsta requested changes Jul 18, 2025
View reviewed changes
Comment thread packages/transport/test/modules/relationships/RelationshipsController.test.ts
Comment thread packages/transport/src/modules/relationships/local/Relationship.ts
@jkoenig134 jkoenig134 requested a review from britsta July 18, 2025 10:45
@jkoenig134 jkoenig134 enabled auto-merge (squash) July 18, 2025 10:45
britsta
britsta approved these changes Jul 18, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@britsta britsta left a comment

Choose a reason for hiding this comment

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

LGTM

@jkoenig134 jkoenig134 merged commit dfc1975 into release/v7 Jul 18, 2025
15 checks passed
@jkoenig134 jkoenig134 deleted the remove-relationship-caches branch July 18, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@britsta britsta britsta approved these changes

@tnotheis tnotheis tnotheis left review comments

@Milena-Czierlinski Milena-Czierlinski Awaiting requested review from Milena-Czierlinski

Assignees

No one assigned

Labels

breaking-change A breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jkoenig134 @tnotheis @britsta