-
Notifications
You must be signed in to change notification settings - Fork 5
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
Support for Diesel 2.0? #2
Comments
Hey, thanks for the offer. I am willing to do this for free since I need it myself too, so I whipped up a branch that you can use until Diesel 2.0 comes out: feat/diesel-2.0. Let me know if you run into anything :) |
Hey @ThouCheese , So far so good. One issue I'm running into on this is supporting the new |
Hey there @phayes, sorry about dropping off the radar for a little there. I've pushed another change that modifies the attributes to look like this: #[derive(Debug, Clone, Copy, PartialEq, Eq, diesel::deserialize::FromSqlRow, DbEnum)]
#[diesel(sql_type = SmallInt)]
#[diesel_enum(error_fn = CustomError::not_found)]
#[diesel_enum(error_type = CustomError)]
pub enum Status {
/// Will be represented as 0.
Ready,
/// Will be represented as 1.
Pending,
} I'm no master of proc macros, so it doesn't produce the best error messages, but let me know what you think! |
I just updated to the branch, things are compiling. I will be able to report back in the near future if things are working correctly. |
With Diesel now at 2.1.0, what are the odds of getting this merged into the current crate? |
Hi there,
I was wondering if there are any plans to support diesel 2? I would be willing to pay for this development if required.
The text was updated successfully, but these errors were encountered: