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

update BatchRecordChangeUpdateResponse #70

Merged

Conversation

britneywright
Copy link
Contributor

Description of the Change

Return full API response after creating a batch change.

Comments string `json:"comments,omitempty"`
OwnerGroupID string `json:"ownerGroupId,omitempty"`
Changes []RecordChange `json:"changes,omitempty"`
ID string `json:"id,omitempty"`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind addressing this formatting issue, please? :)

Comments string `json:"comments,omitempty"`
OwnerGroupID string `json:"ownerGroupId,omitempty"`
Changes []RecordChange `json:"changes,omitempty"`
ID string `json:"id,omitempty"`
Copy link
Contributor Author

@britneywright britneywright Nov 15, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so this looks like it has extra indentation here, but when i view the file it shows the ID attribute aligned with everything else.

diff view:
Screen Shot 2019-11-15 at 3 27 40 PM

file view:
Screen Shot 2019-11-15 at 3 27 58 PM

@britneywright britneywright force-pushed the update-BatchRecordChangeUpdateResponse branch from b31ffe8 to ae3261f Compare November 15, 2019 21:16
@mdb mdb merged commit a3af7b3 into vinyldns:master Nov 18, 2019
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