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

replace interface{} with string for zms-cli import operations #2391

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

havetisyan
Copy link
Collaborator

@havetisyan havetisyan commented Nov 4, 2023

Description

addresses zms-cli panic with:

panic: interface conversion: interface {} is map[string]interface {}, not map[interface {}]interface {}

error message when updating a domain

Contribution Checklist:

  • The pull request does not introduce any breaking changes
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Attach Screenshots (Optional)

Signed-off-by: Henry Avetisyan <hga@yahooinc.com>
@abvaidya abvaidya merged commit ceaf400 into master Nov 6, 2023
2 checks passed
@abvaidya abvaidya deleted the zmscli branch November 6, 2023 19:28
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

Successfully merging this pull request may close these issues.

2 participants