Skip to content
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 kavenegar.d.ts for typescript support #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Hesamp
Copy link

@Hesamp Hesamp commented Nov 10, 2020

این فایل مناسب زمانی هست که از زبان تایپ اسکریپت استفاده کنیم

@molaeiali
Copy link

Looks like no one cares about this project since 2018

status: number,
statustext: string,
sender: string,
receptor: string,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should change this to,

receptor: string[] || string

because in some cases we can also pass array of numbers and send to multiple phone numbers, this is also not a good idea because it takes so long even for small number of receptor items

@jexroid
Copy link

jexroid commented Aug 25, 2024

this codebase is like the weakest and outdated package that Microsoft ever stored in npm repo... i hope they merge some good pr like this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants