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

Fix trait errors with automerge #31

Closed
gterzian opened this issue Jul 6, 2023 · 3 comments · Fixed by #33
Closed

Fix trait errors with automerge #31

gterzian opened this issue Jul 6, 2023 · 3 comments · Fixed by #33

Comments

@gterzian
Copy link
Contributor

gterzian commented Jul 6, 2023

With automerge 0.5, when copying some structs from the example, I get errors like:

41  | #[derive(Debug, Clone, Reconcile, Hydrate, PartialEq)]
    |                                   ^^^^^^^ the trait `AsRef<automerge::exid::ExId>` is not implemented for `ObjId`
42  | struct Address {
43  |    line_one: String,
    |    ---------------- required by a bound introduced by this call

This goes away when going back to automerge 0.4

@bobby
Copy link
Contributor

bobby commented Jul 21, 2023

I'm having the same issue. Our project needs some of the new features in automerge 0.5, including the new Patch API that replaces some of the older Observable API.

I started a PR to address this issue (the branch), but quickly became out of my depth trying to resolve Rust compiler issues related to automerge API changes.

I'm happy to help with this if needed.

@alexjg
Copy link
Collaborator

alexjg commented Jul 21, 2023

This is published as autosurgeon 0.8.0

@bobby
Copy link
Contributor

bobby commented Jul 21, 2023

Kudos autosurgeon team, your turnaround time on issues is incredible!

@gterzian gterzian changed the title Fix for use with automerge 0.5 Fix trait errors with automerge Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants