-
Notifications
You must be signed in to change notification settings - Fork 45
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
meraki_network constraints around local and remote status pages is incorrect #437
Comments
I've already identified the issue in meraki_network and test coverage and am happy to submit a PR for review. |
@kbreit - Question: what's the story with meraki_network and meraki_network_settings? the issue is in both. |
Meraki removed the settings from the network endpoint and moved it to a network settings API endpoint so I recently created the The API documentation states
I'm enforcing the policy in the playbook but am considering removing that rule since I don't enforce every constraint the API imposes, so it's quite inconsistent. |
Regarding the |
@kbreit - just realised i might have miss-understood you there and I've raised a PR you intended to do. happy for you to reject if thats the case :-) . |
With:
... the following error is seen:
Logic in meraki_network is incorrect, as is the test-case when compared to the GUI. as a side note, the API will accept local=false, remote=true, even if this is not supported by the dashboard GUI.
The text was updated successfully, but these errors were encountered: