Skip to content

Remove Message caches#665

Merged
jkoenig134 merged 20 commits into
release/v7from
remove-message-caches
Jul 18, 2025
Merged

Remove Message caches#665
jkoenig134 merged 20 commits into
release/v7from
remove-message-caches

Conversation

@jkoenig134
Copy link
Copy Markdown
Contributor

@jkoenig134 jkoenig134 commented Jul 17, 2025

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 17, 2025
@jkoenig134 jkoenig134 changed the base branch from main to release/v7 July 17, 2025 11:25
@jkoenig134 jkoenig134 marked this pull request as ready for review July 17, 2025 14:41
@britsta britsta changed the title Remove message caches Remove Message caches Jul 17, 2025
Comment thread packages/transport/src/modules/messages/local/Message.ts
Comment thread packages/transport/src/modules/sync/migrations/DeviceMigrations.ts Outdated
Comment thread packages/transport/src/modules/messages/local/Message.ts
Comment thread packages/transport/src/modules/sync/migrations/DeviceMigrations.ts Outdated
Copy link
Copy Markdown
Member

@Milena-Czierlinski Milena-Czierlinski left a comment

Choose a reason for hiding this comment

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

LGTM, but we should probably merge #667 before. :)

@jkoenig134
Copy link
Copy Markdown
Contributor Author

LGTM, but we should probably merge #667 before. :)

Yep!

britsta
britsta previously approved these changes Jul 17, 2025
tnotheis
tnotheis previously approved these changes Jul 18, 2025
@jkoenig134 jkoenig134 merged commit c4a71c9 into release/v7 Jul 18, 2025
15 checks passed
@jkoenig134 jkoenig134 deleted the remove-message-caches branch July 18, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change A breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants