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

docs/nia: api updates #9833

Merged
merged 2 commits into from
Feb 25, 2021
Merged

docs/nia: api updates #9833

merged 2 commits into from
Feb 25, 2021

Conversation

lornasong
Copy link
Member

Below are some updates to the apis that resulted out of work on the disable/enable cli

Commits: (more details in message)

  • Returning an error object instead of error string across all apis (here)
  • Support new "unknown" status value and new "enabled" field in the status api
    docs and examples (here)

 - Reorganize Overall Status response to return status information as a sub-
 structure of task_summary
 - Add new 'unknown' status value to Overall Status response
 - Add new enabled information as a substructure of task_summary
 - Add new enabled field to Task Status
@lornasong lornasong added type/docs Documentation needs to be created/updated/clarified theme/consul-terraform-sync Relating to Consul Terraform Sync and Network Infrastructure Automation labels Feb 25, 2021
@lornasong lornasong requested a review from a team February 25, 2021 16:06
Copy link
Contributor

@eikenb eikenb left a comment

Choose a reason for hiding this comment

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

👍

@lornasong lornasong merged commit 36221e4 into nia/docs-0.1.0-beta Feb 25, 2021
@lornasong lornasong deleted the nia-api-updates branch February 25, 2021 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/consul-terraform-sync Relating to Consul Terraform Sync and Network Infrastructure Automation type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants