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

[Bug] CLI link to telemetry page 404s #2339

Closed
antross opened this issue Apr 30, 2019 · 1 comment

Comments

Projects
None yet
2 participants
@antross
Copy link
Member

commented Apr 30, 2019

The CLI links to:

The actual documentation is at:

Regressed by 8ad4516

I'd prefer to make the original link work again if possible, but I'm not sure how this might interact with the table-of-contents generation used for the docs.

@molant

This comment has been minimized.

Copy link
Member

commented May 2, 2019

I'd prefer to make the original link work again if possible, but I'm not sure how this might interact with the table-of-contents generation used for the docs.

We should have some type of redirect in place. The problem with our current setup is that we have IIS in front and we are splitting the traffic between docs to files and the server/search to the node part.

Some options:

  • Serve everything via node
  • Migrate to Azure WebApp with Linux. It's a single container and we could add nginx in front of us

Either way I think that should be the scope of another issue. I'll be happy if we just update to the new link.

karansapolia added a commit to karansapolia/hint that referenced this issue May 7, 2019

@karansapolia karansapolia referenced this issue May 7, 2019

Merged

Fix: Correct CLI link to telemetry page #2390

2 of 2 tasks complete

@molant molant closed this in #2390 May 7, 2019

molant added a commit that referenced this issue May 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.