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: move folders/files metadata out of Folders entries #1283
Feat: move folders/files metadata out of Folders entries #1283
Conversation
16d73ad
to
5c3bbb2
Compare
5c3bbb2
to
b7f2036
Compare
b7f2036
to
3648f53
Compare
3648f53
to
37999d3
Compare
9829fbb
to
a60e10c
Compare
b22d42e
to
cbe3156
Compare
92ce84f
to
1ce392f
Compare
1ce392f
to
5c34bd3
Compare
fn add_entry(&mut self, name: OsString, content: FolderEntry) -> Result<(Metadata, XorName)> { | ||
// create metadata Chunk for this entry | ||
let metadata = Metadata { | ||
name: name.to_str().unwrap_or("unknown").to_string(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be just the OsString
? The previous todo said the following
// TODO: conversion to String will be removed when metadata is moved out of the Register
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a problem with wasm build and unsafe api from OsString, it needs more investigation
This also makes it possible to sync/compare local changes with remote version of folders/files.
Requires PR #1304 .
Requires PR #1311 .
Description
reviewpad:summary