Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve docs on Interceptors #3218

Open
tastypackets opened this issue May 8, 2024 · 1 comment
Open

Improve docs on Interceptors #3218

tastypackets opened this issue May 8, 2024 · 1 comment
Labels
Docs Changes related to the documentation enhancement New feature or request

Comments

@tastypackets
Copy link
Contributor

tastypackets commented May 8, 2024

This would solve...

Currently docs have limited information for interceptors and it's not easy to discover them, Dispatcher.compose() is probably the best documentation in the docs site and it lacks an example or information on the handler callbacks. There is what looks like an outdated DispatchInterceptor markdown file in the docs, which is no longer part of the docs sidebar or links. I think this is left over from the interceptors param on Client that's deprecated?

The implementation should look like...

Add more detailed documentation on interceptors, ideally it should be easier to discover and include handlers examples and or documentation on the handlers.

One option might be to link the handlers documented above on the dispatch method, since they are well documented there. I think right now the issue is more so discovery of interceptors and handlers. How can we better surface the information in the docs. 🤔

I have also considered...

Additional context

If this issue is still open for a bit I can try and open a PR to help expand the docs on interceptors, but a little busy this week and wanted to open the issue to make sure it's tracked. 🙂

@tastypackets tastypackets added the enhancement New feature or request label May 8, 2024
@metcoder95 metcoder95 added the Docs Changes related to the documentation label May 8, 2024
@metcoder95
Copy link
Member

You are right, the DispatchInterceptro is a concept that will be dropped in next major, so currently its usage is not recommended but Dispatcher.compose instead.

Will be nice to have an enhancement on the documentation, we are aware we need to support in there.

I believe we can start with an example of an interceptor with custom handlers and link to the handler's documentation, but open to suggestions 🙂

A PR is always welcomed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Changes related to the documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants