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

Fix SIP plugin missed_call event call_id field name typo #2703

Merged
merged 1 commit into from
Jun 14, 2021

Conversation

ycherniavskyi
Copy link
Contributor

In all events except missed_call used call_id field name for SIP Call-ID. And even docs of missed_call points call_id field name. So callid seems to be a typo here.

@lminiero
Copy link
Member

Fair point, it looks like the name of the property was changed in #2679 (my fault for not catching that). Mentioning @nicolasduteil who authored the PR, and so may need to be aware the property name is being changed (in case he uses it somewhere).

@lminiero lminiero merged commit a467088 into meetecho:master Jun 14, 2021
@nicolasduteil
Copy link
Contributor

Yep, my mistake. Sorry about that

@lminiero
Copy link
Member

No worries! Just wanted you to be aware of the new change.

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

Successfully merging this pull request may close these issues.

3 participants