-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add Missing Methods for Notification templates #69
Comments
Hi Cliftonz, swagger shows that endpoints under notification templates have been renamed to Workflows. Under Workflows there are 6 endpoints and all of them are implemented under NotificationTemplateApi. Trying out all 6 endpoints either responses with 401 or 403. |
@FazleRabbbiferdaus172 where are you trying out the endpoints? |
@unicodeveloper Swagger link is provided in the issue description, I tried out the endpoints under the Workflow section. |
Can you try them out in the api reference doc? https://docs.novu.co/api-reference/workflows/get-workflows Add your API key so you can get the results. |
@unicodeveloper Tried them out in api reference doc, Get workflows endpoint responses in 200. But do the reference docs endpoints ensure that the python sdk wroks? |
@FazleRabbbiferdaus172 I am not sure if i get you correctly. But you can always use the reference docs to test any of the endpoints quickly. It's independent of the SDK. |
Some endpoints of the following section can't be hit from this SDK.
We would like to ensure that all endpoints can be hit. For this issue, we would like to build out the Notification templates
Some of these methods may already exist. If so no need to change/update them, just point out that they already exist and we will close the issue. This is just to bring every SDK we have on par with all the methods available.
Check all the endpoints here and add the missing ones.
The text was updated successfully, but these errors were encountered: