-
Notifications
You must be signed in to change notification settings - Fork 7
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
Is this documentation a joke ? #25
Comments
Jokes are for laughs, this documentation is a shame, it makes you want to change everything on the spot |
In each new project you need to open at least ten tabs and spend 4 hours just to make something "simple" work |
I tried that, but it's not going well so far 😅 #19 |
This should work - example with ContactsApi:
|
Glad someone said that cuz in 5 minutes of signing up i had just thought the same... not to meniton what the hell was in the mind of the person who created this shit? zero design patterns completelly crazy mess |
This works for me:
Documentation is absolutely awful it's ridiculous. Incorrect and out of date information everywhere. |
Such a waste of time! Documentation sucks! Going back to Sendgrid. |
Totally disappointed with this documentation! |
Hello! Please take a look and hope it helps. |
Yeah, this was a waste if time today |
I think it was still wrong. How API key could be setup should be as follows:
|
Such a joke. This one of the worst docs I've seen. How can you push new SDK with this shit of a documentation!!?? |
It takes 5 minutes to update those TS errors. Why isn't anyone doing it? Any manager seeing this should realize that this is bad for the business when dozens of developers complain that there are errors in README examples. For anyone looking for correct ES6 imports without TS errors, you can do this: import { TransactionalEmailsApi, SendSmtpEmail, TransactionalEmailsApiApiKeys } from '@getbrevo/brevo'
const apiInstance = new TransactionalEmailsApi()
apiInstance.setApiKey(
TransactionalEmailsApiApiKeys.apiKey,
'<your-api-key>'
)
const sendSmtpEmail = new SendSmtpEmail() And from there the rest of the code in the README will work. |
We're still maintaining a version at @groupcards/brevo for anyone interested. See #19 for more details. Also means you don't have to create new classes for every part of the api.
|
This doesn't seem to work either. |
Ah, looks like there was an issue with the build that meant to had to add |
Yes it works after updating to the new version. Thank you. |
@alexbrazier I'm still getting "This expression is not constructable." |
This is what I have:
And this is the error I get:
Is anyone else getting this error? |
Hi @ishaangandhi
This is not reproducible every time, but it occurred with me just once and I followed this above steps to fix it. |
There is only a single feature, that does not even work. Authentication is a protected key.
I'm sorry, but the team is obviousely not taking seriously the project or even involved in it.
We just have to compare this sh*t to the golang sdk documentation to say so.
Even worst, commits are like "update sdk". Seriously ?
The text was updated successfully, but these errors were encountered: