mohs8421

Results 34 comments of mohs8421

regarding 1. the original SQLx error is preserved anyway using the `#[source]` tag. 2. that is an acceptable reason. 3. that is only possible with the knowledge of the database...

@Sculas that was my original intention with this discussion and the pull request in general. This pull request is now merely laying the foundation for what you suggest. There will...

shortly regarding the name of errors: I think in `UniqueConstraintError` the `Error` suffix seems to be noisy in my opinion, as it is already within the `DbErr` enum. I would...

After some searching, I found a solution, which looks like this: ```rust pub fn time() -> i64 { let mut timer: time_t = 0; unsafe { esp_idf_sys::time(&mut timer); } timer...

> @mohs8421 hello! Thank you for the PR, some tests have failed. I saw that, but didn't have time to look deeper into these so far. Besides this change is...

that means going for the "more sophisticated variant", this probably means more enum variants, it might not necessarily be more difficult than this approach. The sqlx error interpreting methods for...

> My main concern is that just throwing out the db depent error isn't ideal. Considering SeaORM can be used on multiple db backend. If I go at an other...

As a first thing I fixed the errors, that had occurred. Now I took a look at the error conversion code again, and the thing is, that if I continue...

> Example of `anyhow` context with downcasting: https://github.com/dtolnay/anyhow/blob/master/tests/test_context.rs#L96-L118 I still don't see how this would make it easier to implement the desired behaviour, can you help me out here so...

Thanks, I guess that article about anyow and thiserror is really helpful on that regard. I will adapt my change to that when I have the opportunity to do this.