mbilling / PBIConnectors

The Microsoft Power BI custom connector support-project
MIT License
71 stars 33 forks source link

Facebook Ads does not load ad accounts in navigator #23

Open spijs opened 4 years ago

spijs commented 4 years ago

I don't see any ad accounts when trying to connect to the data source. When in Facebook adsmanager I see at least 2 personal and one business account. Have I misunderstood any concepts here or could you guide me to any relevant tests I should run?

Thanks. image

Update: When running the https://graph.facebook.com/v6.0/me/adaccounts call using Postman, I do see the three accounts. At least one of them returns data on /ads.

spijs commented 4 years ago

Update: When I limit the 'insights' to all only (not the active ones), everything seems to work.

GuidoPinaresCh commented 4 years ago

Could you resolve it?, i get this two errors first when i try to load into power bi

image

and second when im trying to compile it

image

Help please!!

mbilling commented 4 years ago

Try and change the function // Navigation entry point [DataSource.Kind="FacebookAds", Publish="FacebookAds.Publish"] shared FacebookAds.Ads = (optional url as text, optional params as record) => let key = Extension.CurrentCredential()[access_token], seed = baseUrl & "me/adaccounts?" & Uri.BuildQueryString([access_token=key,fields="id,name",limit="1"]), pagedResult = FacebookAds.GetPagedData(seed, {}), table = FacebookAds.AdAccountsToNavigationTable(pagedResult) in table;

to

// Navigation entry point [DataSource.Kind="FacebookAds", Publish="FacebookAds.Publish"] shared FacebookAds.Ads = (optional url as text, optional params as record) => let key = Extension.CurrentCredential()[access_token], seed = baseUrl & "me/adaccounts?" & Uri.BuildQueryString([access_token=key,fields="id,name",limit="1"]), pagedResult = FacebookAds.GetPagedData(seed, {}), table = FacebookAds.AdAccountsToNavigationTable(pagedResult) in seed;

Check what you get from FB by pasting the URL into a browser I looks like multiple of the same account - I haven't encountered this before You would need to distinct the list returned if it contains duplicates on the same ad account

GuidoPinaresCh commented 4 years ago

Thanks for the support, but now i get another error.

[cid:a90855a4-73ba-4ead-a1f2-4e89ea29d756]

Could you please help me? thanks a lot!


De: Michael Frost Billing notifications@github.com Enviado: miércoles, 8 de abril de 2020 3:53 Para: mbilling/PBIConnectors PBIConnectors@noreply.github.com Cc: GUIDO PINARES CHIRINOS 46936526@ucsm.edu.pe; Comment comment@noreply.github.com Asunto: Re: [mbilling/PBIConnectors] Facebook Ads does not load ad accounts in navigator (#23)

Try and change the function // Navigation entry point [DataSource.Kind="FacebookAds", Publish="FacebookAds.Publish"] shared FacebookAds.Ads = (optional url as text, optional params as record) => let key = Extension.CurrentCredential()[access_token], seed = baseUrl & "me/adaccounts?" & Uri.BuildQueryString([access_token=key,fields="id,name",limit="1"]), pagedResult = FacebookAds.GetPagedData(seed, {}), table = FacebookAds.AdAccountsToNavigationTable(pagedResult) in table;

to

// Navigation entry point [DataSource.Kind="FacebookAds", Publish="FacebookAds.Publish"] shared FacebookAds.Ads = (optional url as text, optional params as record) => let key = Extension.CurrentCredential()[access_token], seed = baseUrl & "me/adaccounts?" & Uri.BuildQueryString([access_token=key,fields="id,name",limit="1"]), pagedResult = FacebookAds.GetPagedData(seed, {}), table = FacebookAds.AdAccountsToNavigationTable(pagedResult) in seed;

Check what you get from FB I looks like multiple of the same account - I haven't encountered this before You would need to distinct the list returned if it contains duplicates on the same ad account

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/mbilling/PBIConnectors/issues/23#issuecomment-610837321, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANE6KGWD6I36MK6N7YQXMKTRLQ3Q5ANCNFSM4K6YZPCA.