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

Fail better when token does not have rights #21

Closed
PLaRoche opened this issue Dec 3, 2018 · 0 comments
Closed

Fail better when token does not have rights #21

PLaRoche opened this issue Dec 3, 2018 · 0 comments

Comments

@PLaRoche
Copy link
Contributor

PLaRoche commented Dec 3, 2018

Currently if you are trying to access a resource or project your API key does not have access to, you simply see:

In Core.php line 100: array_merge(): Argument #2 is not an array

See #20 for detailed discovery on

webbtj added a commit to webbtj/manifold-laravel that referenced this issue Dec 9, 2018
…efore trying to merge into the configs array, do not attempt merge if it's not an array
PLaRoche pushed a commit that referenced this issue Dec 11, 2018
* addressing #22 - test parsed api response format as array before testing count()

* address #21 - test the results of loading an api resource before trying to merge into the configs array, do not attempt merge if it's not an array
PLaRoche pushed a commit that referenced this issue Dec 12, 2018
* addressing #22 - test parsed api response format as array before testing count()
* address #21 - test the results of loading an api resource before trying to merge into the configs array, do not attempt merge if it's not an array
* reworking manifold config and caching system to allow manifold populated configs to be available throughout other config files
* fix DiskCache has method, a true return should require the expiration is in the *future*, not the past
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

1 participant