Skip to content

Not all traits implemented #375

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

Open
DennohKim opened this issue Aug 5, 2024 · 12 comments
Open

Not all traits implemented #375

DennohKim opened this issue Aug 5, 2024 · 12 comments

Comments

@DennohKim
Copy link

Error

How can i resolve this error when boostrapping create-rust-app?

@rivertam
Copy link

rivertam commented Aug 8, 2024

Getting this too

@Adamkaram
Copy link

me either

@gitbobobo
Copy link

modify Cargo.toml can fix this issue, but more errors are coming...😭

[dependencies.diesel]
default-features = false
features = ["postgres", "r2d2", "chrono"]
version = "=2.1"

@pegasusplus
Copy link

continue modify Cargo.toml, lock version of "dsync" to "0.0.16", then "cargo fullstack" is OK now.

@pegasusplus
Copy link

Now runtime panik:

Starting frontend server @ http://localhost:21012/
thread 'tokio-runtime-worker' panicked at /Users/zhenpingzhao/.cargo/registry/src/index.crates.io-6f17d22bba15001f/create-rust-app-11.0.0/src/dev/frontend_dev_server.rs:29:10:
called Result::unwrap() on an Err value: Os { code: 2, kind: NotFound, message: "No such file or directory" }
stack backtrace:

Need more config file?

@ideakaran
Copy link

I am also facing the issue. I also tried with dsync = "0.0.16" but did not work.

.cargo/registry/src/index.crates.io-6f17d22bba15001f/diesel_logger-0.3.0/src/lib.rs:80:1
|
80 | / impl Connection for LoggingConnection
81 | | where
82 | | C: Connection + 'static,
83 | | ::Backend: std::default::Default,
84 | | <C::Backend as Backend>::QueryBuilder: Default,
| |___________________________________________________^ missing instrumentation, set_instrumentation in implementation

@AnthonyMichaelTDM
Copy link
Collaborator

Seems to be an issue with https://github.com/shssoichiro/diesel-logger, I'll see if I can dig into this a bit more

@elanora96
Copy link

elanora96 commented Nov 15, 2024

Thanks to the merged commit above, diesel_logger seems to have been fixed. Unfortunately a new release has not been made yet so modifying Cargo.toml to use the shssoichiro/diesel-logger master branch (or the commit branch) is necessary

edit: v0.4.0 released

Cargo.toml

[patch.crates-io]
diesel_logger = "0.4.0"

Unfortunately there seems to be a separate issue with dsync v0.1.0 delivering some breaking changes to the initial file in .cargo/bin/dsync.rs 😩. I don't believe that is linked to this issue but figured I would mention it here before creating a separate issue as dsync was brought up earlier.

@rootCircle
Copy link

Since this error has been patched upstream we can upgrade diesel-logger to 0.4.0 and it should work as expected!

Wulf added a commit that referenced this issue Mar 10, 2025
@Wulf
Copy link
Owner

Wulf commented Mar 10, 2025

Thanks for sharing folks!

Is this all we need?

#379

@Florian9M
Copy link

How can we solve this problem ?

@elanora96
Copy link

There were quite a few issues in the default template with the later releases of dsync. I started working on it in my fork elanora96/create-rust-app, but admittedly I ended up not using create-rust-app and abandoned it. Fair warning I was also less experienced with Rust last time I checked this out :p

I can take another look at it if help is needed, though I cannot necessarily commit to any timeline

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

No branches or pull requests