Skip to content

Deprecate the Logs list API - #65

Merged
brandonb927 merged 1 commit into
masterfrom
deprecate/logs-list-api
Aug 4, 2017
Merged

Deprecate the Logs list API#65
brandonb927 merged 1 commit into
masterfrom
deprecate/logs-list-api

Conversation

@brandonb927

Copy link
Copy Markdown

Deprecating the Log List API endpoint.

Description

  • Update README to reflect Log List API changes
  • Remove Log List API support from library

Motivation and Context

The endpoint has been deprecated and will later be removed. We are updating the client to reflect this.

How Has This Been Tested?

Unit tests have been removed that rely on Log List API code.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@brandonb927 brandonb927 self-assigned this Aug 2, 2017
@brandonb927
brandonb927 requested a review from demoore August 2, 2017 23:53
@demoore

demoore commented Aug 4, 2017

Copy link
Copy Markdown
Member

This looks great. Can you also fix the failing customer and version tests in this PR?

@brandonb927
brandonb927 merged commit 8bf7a5c into master Aug 4, 2017
@brandonb927
brandonb927 deleted the deprecate/logs-list-api branch August 4, 2017 23:44
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