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
KAZOO-5800: Re-check for call restrictions after shortdial correction #4675
Conversation
f2eb69e
to
3e6bc17
Compare
Can you rebase and recheck this functionality? Let's get it merged once rebased |
Done :) |
{'error', _R} -> | ||
Call; | ||
{'ok', JObj} -> | ||
CallRestriction = kz_json:get_value(<<"call_restriction">>, JObj), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use the more specific getters when you can: get_json_value/2
(maybe with a default kz_json:new()
?) and get_ne_binary_value/2
on the next line.
@@ -77,6 +77,9 @@ | |||
"Custom-SIP-Headers": { | |||
"type": "object" | |||
}, | |||
"Denied-Call-Restrictions": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are they really a string? Or should this be an object? You can hint at it in the kapi module using a validator or you can manually correct this schema.
0684268
to
75c9e48
Compare
b5046cd
to
b5cff06
Compare
@harenson i refactored a bit; mind checking if things are still working on your tests? Will merge if you agree :) |
…#4675) refactor a bit refactor to named function fix function arity
b5cff06
to
77f0291
Compare
No description provided.