xtra icon indicating copy to clipboard operation
xtra copied to clipboard

Introduce `SendFuture::repeat_every`

Open thomaseizinger opened this issue 3 years ago • 3 comments

Extracted out of #71.

This should be introduced via an extension crate to not grow the core API of xtra unnecessarily.

It is important that this must work only on Handlers that return (). That is also the main reason for providing this functionality to users and not telling them to write it themselves. It is a potential footgun to drop that return value.

thomaseizinger avatar Jul 14 '22 21:07 thomaseizinger

This sounds like a nice API! Though, as we are currently moving toward removing convenience methods to slim down the API with a view to an extension crate soon, then this might be an odd addition to merge now

Restioson avatar Jul 15 '22 13:07 Restioson

Fair point. Would you be okay with removing the feature entirely and adding it back later once the dust around the slimming down has settled?

thomaseizinger avatar Jul 15 '22 13:07 thomaseizinger

Sure, that makes sense!

Restioson avatar Jul 15 '22 13:07 Restioson