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

The AdAccountFields enum does not currently implement the ACCOUNT_STATUS field. #588

Closed
iamyukihiro opened this issue Oct 27, 2023 · 1 comment

Comments

@iamyukihiro
Copy link

It appears that the AdAccountFields enum does not currently implement the ACCOUNT_STATUS field. When examining the API response, it provides values such as "0" and "1," which are equivalent to the values in FacebookAds\Object\Values\AdStatusValues and FacebookAds\Object\Values\CampaignStatusValues . However, there is no implementation for FacebookAds\Object\Values\AdAccountStatusValues , which can be confusing for developers as it lacks explicit representation.

I kindly request that you consider implementing FacebookAds\Object\Values\AdAccountStatusValues to provide clarity and consistency for developers. Your attention to this matter would be greatly appreciated.

@stcheng
Copy link
Contributor

stcheng commented Jan 19, 2024

Currently, we are using the 'unsigned int' type to indicate account status, while for campaigns, ads, and similar implementations, we are utilizing an enum. Therefore, implementing this change will pose a limitation for us.

@stcheng stcheng closed this as completed Jan 19, 2024
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

No branches or pull requests

2 participants