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

fix: enable interceptor for REST interface on DWH #1826

Merged
merged 1 commit into from
Jan 16, 2019

Conversation

nikonov1101
Copy link
Member

@nikonov1101 nikonov1101 commented Jan 14, 2019

At least we also should notify a user about the "out of sync" state.

@nikonov1101 nikonov1101 requested a review from a team as a code owner January 14, 2019 16:19
@nikonov1101 nikonov1101 added 💩 T: bug This PR/Issue fixes a bug S: DWH This PR/Issue changes DWH P: low This PR/Issue has minor priority labels Jan 14, 2019
@3Hren
Copy link
Member

3Hren commented Jan 15, 2019

Seems no reason for chaining.

@nikonov1101 nikonov1101 force-pushed the fix/dwh/rest-interceptor branch from 991b13e to dd6631c Compare January 15, 2019 11:27
@nikonov1101
Copy link
Member Author

Drop PR chaining.
@antmat corresponding changes made in #1824

@3Hren 3Hren merged commit 2ec17d7 into master Jan 16, 2019
@3Hren 3Hren deleted the fix/dwh/rest-interceptor branch January 16, 2019 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P: low This PR/Issue has minor priority S: DWH This PR/Issue changes DWH 💩 T: bug This PR/Issue fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants