ekazaev/route-composer

Make NavigationDelayingInterceptor context generic

WingedDoom opened this issue · 2 comments

Method .adding(...) requires that RI.Context == C, which means factory and interceptor context types should be equal.

This, however, is not always possible with current implementation of NavigationDelayingInterceptor, which has Context = Any?.

Adding a generic parameter Context to NavigationDelayingInterceptor would fix the problem, and it seems easy to do, though I'm not sure how difficult it would be to refactor usages of the interceptor.

@WingedDoom Good catch. Thank you

@WingedDoom I have only one question. I did not expect that this interceptor could be applied to the particular chain of steps. I expected it to be applied to the router once as if that issue exists - it is global to the application.
What is the situation that you want to add it to the one particular configuration rather than the whole router?