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

Typescript experiment #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ahmad-reza619
Copy link

@ahmad-reza619 ahmad-reza619 commented Mar 4, 2024

I've added typescript support for now. but i have no idea how to test if it still works or not

@shaunbharat
Copy link
Owner

Thanks! I'll see if I can get time to test it but I don't know at the moment. Anyway, there aren't formal tests for this library right now, but I think by running the example script after importing the local TypeScript files, we can see if things work.

I am almost certain that something is broken though because it's been 2 years, and Google's UI has almost definitely changed.

@ahmad-reza619
Copy link
Author

I tried just that and yeah we got issues when logging in the account. i'll try to fix the issue for now.

I also added an env so that anyone can just fill in their google credentials for testing without getting it accidentally committed, it would be bad if your credential can be read from git history 😅

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.

2 participants