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 needs to pass option params to API calls #454

Closed
peterbourgon opened this issue Sep 7, 2015 · 1 comment
Closed

UI needs to pass option params to API calls #454

peterbourgon opened this issue Sep 7, 2015 · 1 comment
Assignees
Labels
bug Broken end user or developer functionality; not working as the developers intended it

Comments

@peterbourgon
Copy link
Contributor

  1. Select Containers view
  2. Select Show system containers
  3. Click on any system container

Expected behavior: call to /api/topology/containers/a0b1c2e3d4?system=show is made (note URL param)

Actual behavior: call without URL param is made, resulting in 404

/cc @tomwilkie this explains the regression

@peterbourgon peterbourgon added the bug Broken end user or developer functionality; not working as the developers intended it label Sep 7, 2015
@tomwilkie
Copy link
Contributor

Nb this is a release blocker.

On Monday, 7 September 2015, Peter Bourgon notifications@github.com wrote:

  1. Select Containers view
  2. Select Show system containers
  3. Click on any system container

Expected behavior: call to /api/topology/containers/a0b1c2e3d4?system=show
is made (note URL param)

Actual behavior: call without URL param is made, resulting in 404

/cc @tomwilkie https://github.com/tomwilkie this explains the regression


Reply to this email directly or view it on GitHub
#454.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Broken end user or developer functionality; not working as the developers intended it
Projects
None yet
Development

No branches or pull requests

3 participants