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

add more info to the zones call #3 #6

Merged
merged 2 commits into from Oct 8, 2018
Merged

Conversation

u5surf
Copy link
Contributor

@u5surf u5surf commented Oct 4, 2018

Description of the Change

I add more info to the zone/zones call

Why Should This Be In The Package

#3 told that it can get more info to call the zone/zones in this API

Benefits

user will be able to more information to call zone/zones command.

Possible Drawbacks

may nothing, but I think it will be a bit of complication output.

Applicable Issues (Optional)

#3 and #4

@CLAassistant
Copy link

CLAassistant commented Oct 4, 2018

CLA assistant check
All committers have signed the CLA.

@mdb
Copy link
Collaborator

mdb commented Oct 8, 2018

@u5surf Thanks! This looks good! We may want to return to better formatting some of this data if it becomes difficult to read at certain screen sizes when there's a lot of data, but this is perfect for now!

@mdb mdb merged commit ff92720 into vinyldns:master Oct 8, 2018
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.

None yet

3 participants