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

doc: Document data model Integration::Hooks #245

Open
sojan-official opened this issue Mar 30, 2022 · 0 comments
Open

doc: Document data model Integration::Hooks #245

sojan-official opened this issue Mar 30, 2022 · 0 comments

Comments

@sojan-official
Copy link
Member

there are two types of hooks ..
account hooks -> inbox id should be nil .. works across the account (for example webhooks, slack etc )
inbox hooks -> inbox id is present, work in inbox scope, multiple ones can be present ( example Dialogflow )

https://github.com/chatwoot/chatwoot/blob/baf697f064bfdd8c691d1c2ee84fff8f8130a699/app/listeners/hook_listener.rb#L22

the particular line of code is to skip the case where we are checking for an inbox hook, but the message event that was triggered doesn't belong to that inbox.
in the case of Dialogflow, the inbox id should be empty for the hook. hence that line of code shouldn't come into effect since hook.inbox = nil

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

No branches or pull requests

1 participant