-
Notifications
You must be signed in to change notification settings - Fork 13
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
feature-request: add typings for typescript support #28
Comments
Hi @mattmeye! Thank you for the suggestion. We haven't planned that previously and since this is not a bug report, I can't promise when/if it will be done. We would gladly accept a PR, though, if you happen to have done it for your needs already. |
@mattmeye you can now find the defintions at https://www.npmjs.com/package/@types/chartmogul-node. Please let us know if you face any issue using it. |
@types/chartmogul-node are outdated. Miss-match right of the bat export class Config {
VERSION: string;
API_BASE: string;
retries: number;
constructor(token: string, secret: string, base?: string);
getAccountToken(): string;
getSecretKey(): string;
} vs class Config {
constructor (apiKey, apiBase) {
this.apiKey = apiKey;
this.apiBase = apiBase || API_BASE;
}
|
see http://definitelytyped.org/
The text was updated successfully, but these errors were encountered: