novuhq / novu-rust

MIT License
17 stars 13 forks source link

Add Missing Methods for Feeds #17

Closed Cliftonz closed 10 months ago

Cliftonz commented 10 months ago

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 Feeds

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.

Aunali321 commented 10 months ago

Hello, i'd like to work on this

unicodeveloper commented 10 months ago

Assigning this to you now @Aunali321

unicodeveloper commented 10 months ago

For more context: these are the endpoints/methods you are building out and you can test them out here by using your API Key.

Aunali321 commented 10 months ago

For more context: these are the endpoints/methods you are building out and you can test them out here by using your API Key.

* Get Feeds: https://docs.novu.co/api-reference/feeds/get-feeds

* Create Feed: https://docs.novu.co/api-reference/feeds/create-feed

* Delete Feed: https://docs.novu.co/api-reference/feeds/delete-feed

I have already implemented some of the methods. how can i test them?

unicodeveloper commented 10 months ago

I am not sure I get what you mean? Whenever you are building features, you test locally to be sure it works on your local rust environment, hits the API and returns the appropriate response.

Don't you have your Rust environment set up?

Aunali321 commented 10 months ago

I mean something like a mock server

unicodeveloper commented 10 months ago

Your features should hit the real Novu API for you to be sure it's working as intended.

It's only when you are writing tests that you need a mock so you are not hitting the real API.