Remove automerge-doc-server and sync Automerge docs from backend using samod
#875
Conversation
jmoggr
force-pushed
the
samod
branch
4 times, most recently
from
06d32e1 to
d3f73eb
Compare
epatters
added
enhancement
epatters
changed the title
automerge-doc-server and serve automerge-repo from backendautomerge-doc-server and sync Automerge docs from backend using samod
epatters
left a comment
epatters
left a comment
There was a problem hiding this comment.
Thanks Jason, this is great! I'm excited to have a simpler architecture and it's beautiful to see all that deleted code.
| use uuid::Uuid; | ||
|
|
||
| /// Insert a JSON value into a map property | ||
| fn insert_value_into_map<'a>( |
There was a problem hiding this comment.
You might have mentioned these helper functions to me before, Jason, but I'm surprised that we have to write them. Is this functionality that you would expect to be upstreamed?
There was a problem hiding this comment.
I was only a bit surprised. It seems like not many people are needing to manipulate Automerge docs both from Javascript and Rust, and practically doing that has only recently been made possible with Samod.
We could be the first people to run into this, I'll open an issue on Samod. It seems reasonable to have this functionality upstreamed.
packages/migrator/src/migrations/m20250805230408_fix_automerge_storage.rs
Show resolved
Hide resolved
jmoggr
force-pushed
the
samod
branch
2 times, most recently
from
b511c1d to
f4ce62a
Compare
No description provided.