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

ui: remove old nspace argument from the proxy instance repository #10039

Merged
merged 2 commits into from
Apr 15, 2021

Conversation

johncowen
Copy link
Contributor

@johncowen johncowen commented Apr 15, 2021

The extra argument meant that the blocking query configuration wasn't
being read properly, and therefore the correct ?index wasn't being sent
with the request.

The error was introduced after our last 1.9.x release (but it is in the 1.10-0-alpha), hence no backport.

@mikemorris are we adding changelog entries for things which were broken in 1.10.0-alpha and fixed here?

The extra argument meant that the blocking query configuratiopn wasn't
being read properly, and therefore the correct ?index wasn't being sent
with the request.
@johncowen johncowen added the theme/ui Anything related to the UI label Apr 15, 2021
@johncowen johncowen added this to the 1.10.0 milestone Apr 15, 2021
@mikemorris
Copy link
Contributor

To clarify, this issue was introduced in #9746, which was not backported to the release/1.9.x branch (as opposed to having been backported but just not yet released), so no backport is necessary.

Changelog entry for beta1 would be good, we'll collapse those down to remove intermediate changes when doing the final 1.10.0 GA release, but helpful for now.

@vercel vercel bot temporarily deployed to Preview – consul April 15, 2021 17:07 Inactive
@vercel vercel bot temporarily deployed to Preview – consul-ui-staging April 15, 2021 17:07 Inactive
@johncowen
Copy link
Contributor Author

Changelog added, just needs an approval now

Copy link
Contributor

@kaxcode kaxcode left a comment

Choose a reason for hiding this comment

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

LGTM

@johncowen johncowen merged commit 76b2122 into master Apr 15, 2021
@johncowen johncowen deleted the ui/bugfix/proxy-instance-cursor branch April 15, 2021 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/ui Anything related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants