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

Extend async webhook context with settings manager #137

Open
krzysztofwolski opened this issue Dec 7, 2022 · 0 comments
Open

Extend async webhook context with settings manager #137

krzysztofwolski opened this issue Dec 7, 2022 · 0 comments
Labels
enhancement New feature or request To refine

Comments

@krzysztofwolski
Copy link
Member

  • webhook handlers most likely will need app settings to process the incoming request. Extend the async webhook context with Settings Manager
  • payload can contain private metadata of the app. The handler factory should be able to find out if that's the case and load it to the Settings manager without the need of the additional request to the API
@krzysztofwolski krzysztofwolski added enhancement New feature or request To refine labels Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request To refine
Projects
None yet
Development

No branches or pull requests

1 participant