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 Google Ads API #499

Closed
davecardwell opened this issue May 23, 2021 · 8 comments · Fixed by #500
Closed

Add Google Ads API #499

davecardwell opened this issue May 23, 2021 · 8 comments · Fixed by #500
Assignees
Labels
enhancement New feature or request

Comments

@davecardwell
Copy link

The Google Ads API has a discovery document available but it is not listed in the googleapis.com API list for some reason.

I can generate the types manually using npm start -- --url https://googleads.googleapis.com/\\\$discovery/rest?version=v7 but couldn’t see any way of adding it to the automated generating/linting/testing/publishing pipeline.

For APIs that are not listed in the API list am I best off adding directly to DefinitelyTyped or is there a simple way to support hard-coded discovery doc URLs in this project?

@Maxim-Mazurok
Copy link
Owner

Hello Dave!
Hm, it's a very good find, thanks for that.
For sure it's better to add this to the CI pipeline.
I will try to add it today.
BTW, where did you find that URL?

@Maxim-Mazurok
Copy link
Owner

@davecardwell I've included googleads API to the CI. And opened PRs to add it to DT.
For now, you can use it from https://www.npmjs.com/package/@maxim_mazurok/gapi.client.googleads
It'll get hourly updates as all other API types from now on.
Let me know if there's anything else I can help you with :)
And please, consider giving this repo a star if you found it useful ;)

@davecardwell
Copy link
Author

@Maxim-Mazurok awesome turnaround, thank you.

I honestly can’t remember where I found the discovery doc originally, sorry!

@Maxim-Mazurok
Copy link
Owner

@davecardwell https://www.npmjs.com/package/@types/gapi.client.googleads just got published, feel free to replace @maxim_mazurok/gapi.client.googleads with @types/gapi.client.googleads. Cheers!

@Maxim-Mazurok
Copy link
Owner

@davecardwell Hello Dave, I see that the current Google Ads API version is 11, and we're still only publishing types for version 7... I'm working on multiple versions support for #652 and just wanted to confirm with you that you are/were using Google Ads on the client-side (front-end)?

@Maxim-Mazurok
Copy link
Owner

Maxim-Mazurok commented Jul 2, 2022

@davecardwell I finally managed to make Google Ads API work on the client using gapi.client.request(), see https://github.com/Maxim-Mazurok/google-tasks-api-ts-example/tree/googleads branch

Still can't make it work using gapi.client.googleads.something() approach tho... Did you use .request() or .googleads approach?

@davecardwell
Copy link
Author

@Maxim-Mazurok I was using it on the front-end. Unfortunately I don’t recall the exact method and I have moved on from that company so no longer have access to the project to check.

@Maxim-Mazurok
Copy link
Owner

I see, thanks anyway for confirming, since it can be used on FE - I'll make sure we keep on generating types for it, cheers!

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

Successfully merging a pull request may close this issue.

2 participants