#484 Iden static

[ikrivosheev]

PR Info

• Closes Using Iden with sqlx is cumbersome #484

New Features

• Create trait IdenStatic

[ikrivosheev]

@nitnelave @tyt2y3 @billy1624 can you review my PR?

[tyt2y3]

Nice addition. I have a feeling like SeaQuery will be slowly more-and-more like SeaORM

[billy1624]

Thanks!! @ikrivosheev

[ikrivosheev]

Nice addition. I have a feeling like SeaQuery will be slowly more-and-more like SeaORM

I think this is good. I would like the SeaQuery to be the best query builder) And SeaORM - ORM and SeaQuery helps it for build query) This is one part of this.

[billy1624]

Hey @tyt2y3, I wonder why we need to move below into SeaQuery? Any good reasons?

pub trait IdenStatic: Iden + Copy + Debug + 'static {
    /// Method to call to get the static string identity
    fn as_str(&self) -> &str;
}

[tyt2y3]

Having two symbols with the same name in the same workspace that works (more or less) the same is confusing.

[ikrivosheev]

@tyt2y3 @billy1624 @nitnelave I have done. Can you have a look at it?