-
Notifications
You must be signed in to change notification settings - Fork 146
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
Recommendation: Any plan on modularising the application/splitting into multiple files and also any plan of converting to typescipt. #75
Comments
Hello, Firstly, me as a maintainer not really interested in this at this point, as long as the code works as is, but most probably, eventually it will be transferred to some other SDK specific developer, and then this decision would be up to them. Secondly, we have quite a low browser support requirements, like IE7 support at this current point, for some legacy software of our clients. So there would be lots of polyfilling needed :) |
@ar2rsawseen Hello, Is there Typescript support among your plans this year? |
Hello, |
Adding Typescript support is something that we would want to do. Currently we can't give any specific timelines of when that would be done. It is possible that it is added by the end of this year. |
I am also interested in TypeScript support. |
At least having some typings is pretty much a no-brainer nowadays. Not seeing even a single .d.ts files is a pretty big no go |
2024 - one of the best open source analytics platform.. no types :( |
Hey thank you! We started adding them but then other SDKs/features took over. We will definitely be adding the types as soon as the availability arises. |
Hey everyone, version 24.4.1 now includes types for the SDK. |
Reason: This would help in attracting more contributors. As it would be easy to understand the code and identify the issues/improvements.
Note: I can contribute in the migration. I did something related to it, splitting into multiple files (Also migrating to Typescript) and now it is working as it should. And it can be further improved for better readability.
The text was updated successfully, but these errors were encountered: