narrowcast/facebook-ads-api — pull request #62 by william-gallop (2014-12-16 17:30:05+00:00)
Title: Breaking change: Currency is optional in reach estimates: treat it so in the code. 

From Facebook docs: "Note that targeting_spec is required for accessing /{account-id}/reachestimate while currency is an optional field. The returned CPC and CPM values are based on the currency specified." https://developers.facebook.com/docs/reference/ads-api/reachestimate/v2.2

We only need the reach estimate, not the CPC and CPM values, and we don't need a currency according to the API, so we shouldn't have to apply one to this method either.

Since this change is breaking anyway, I made the json encoding convention consistent with the rest of this file: to use json.dumps on the inside, instead of relying on the caller to do it.

Note that Facebook are removing the Action Spec with 2.2, so callers of this method will need to visit this anyway.