Skip to content

Implement LoaderTrait for [&M]#2614

Closed
Huliiiiii wants to merge 2 commits intoSeaQL:masterfrom
Huliiiiii:loader
Closed

Implement LoaderTrait for [&M]#2614
Huliiiiii wants to merge 2 commits intoSeaQL:masterfrom
Huliiiiii:loader

Conversation

@Huliiiiii
Copy link
Copy Markdown
Member

PR Info

  • Closes
  • Dependencies:
  • Dependents:

New Features

  • Implement LoaderTrait for [&M], allows LoaderTrait methods to be called on Vec<&M>

Bug Fixes

Breaking Changes

Changes

@Huliiiiii
Copy link
Copy Markdown
Member Author

Huliiiiii commented May 30, 2025

Another way is

impl<T> ModelTrait for &T
where
T: ModelTrait

However, this is not safe enough to call the delete method. (But why is the delete method on ModelTrait?)

@tyt2y3
Copy link
Copy Markdown
Member

tyt2y3 commented Jun 29, 2025

However, this is not safe enough to call the delete method

I don't understand this, where is delete() called?

@Huliiiiii
Copy link
Copy Markdown
Member Author

However, this is not safe enough to call the delete method

I don't understand this, where is delete() called?

#2548 (reply in thread)

@Huliiiiii Huliiiiii closed this Dec 24, 2025
@Huliiiiii Huliiiiii deleted the loader branch December 24, 2025 20:26
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 this pull request may close these issues.

2 participants