Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: media manager v1 #969

Merged
merged 4 commits into from
Nov 20, 2024
Merged

feat: media manager v1 #969

merged 4 commits into from
Nov 20, 2024

Conversation

EvanHahn
Copy link
Contributor

Combines #940, #957, and #956.

I plan to YOLO-merge this if CI passes.

gmaclennan and others added 4 commits November 6, 2024 19:26
…)` (#940)

- Removes `blobStore.download()` and the `LiveDownload` class.
- Adds `blobStore.createEntriesReadableStream()` which is a stream of
  all entries in the blob store, filtered by blob type and variant(s).
- Changes behaviour of `BlobStore` to auto-download blobs according to a
  `downloadFilter`.
- Changes the `BlobStore` filter according to `isArchiveDevice` setting.
- Changes the behaviour of `CoreManager` to not auto-download blob cores
  (this responsibility is now with `BlobStore`).

Closes [#681].

[#681]: #681

Co-authored-by: Evan Hahn <[email protected]>
This is a types-only change. We were already importing the `BlobFilter`
type above, so let's use it!
*You may wish to [review this squashed commit as separate commits][0].*

When you receive a blob filter from another peer, this updates their
sync states. For example, if you receive a blob filter that says "I only
want photo thumbnails", that peer's "wants" bitfield will be reduced.

Closes [#682] and [#905].

[0]: https://github.com/digidem/comapeo-core/pull/956/commits
[#682]: #682
[#905]: #905
@EvanHahn EvanHahn enabled auto-merge (squash) November 20, 2024 15:01
@EvanHahn EvanHahn disabled auto-merge November 20, 2024 15:01
@EvanHahn EvanHahn merged commit 3d1c94b into main Nov 20, 2024
9 checks passed
@EvanHahn EvanHahn deleted the media-manager-v1 branch November 20, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants