novu-java icon indicating copy to clipboard operation
novu-java copied to clipboard

Outdated usage documentation for version `v1.4.0`

Open shygnome opened this issue 2 years ago • 3 comments

It seems the examples in the usage documentation are different from the actual SDK in version v1.4.0. Any plan to update the documentation?

shygnome avatar Oct 23 '23 07:10 shygnome

Hi @shygnome, you're talking about the Usage section in this repo's README not the documentation hosted on novu.co, right?

m-a-r-1912 avatar Oct 25 '23 05:10 m-a-r-1912

Hi @shygnome, you're talking about the Usage section in this repo's README not the documentation hosted on novu.co, right?

I think both this repo's README and the one hosted in docs.novu.co are same. Like in this API reference page https://docs.novu.co/api-reference/events/trigger-event. The java example one still using HashMap object. We are supposed to use the TriggerEventRequest and other request class, right? @m-a-r-1912

shygnome avatar Oct 25 '23 06:10 shygnome

Yeah, according to this repo, the triggerEvent method accepts a TriggerEventRequest as parameter. So any documentation using HashMap type as parameter is wrong. We might need to create a issue(s) for the Docs repo as well. Since that's the code responsible for the api-reference pages.

FYI, it appears that one of the documentation pages has a few up to date examples https://docs.novu.co/quickstarts/java. Potentially some of the examples from there could simply be copied over to the out of date docs.

Edit: I don't know if Novu maintainers already have plans to do some kind of documentation update. Probably a good idea to get feedback from them.

m-a-r-1912 avatar Oct 25 '23 07:10 m-a-r-1912

Closing this as resolved

mayorjay avatar Jun 16 '24 18:06 mayorjay