Skip to content

Conversation

@Rick-Anderson
Copy link
Contributor

@Rick-Anderson Rick-Anderson commented Nov 18, 2024

Review URL

fixes #34167


Internal previews

📄 File 🔗 Preview link
aspnetcore/toc.yml aspnetcore/toc

@Rick-Anderson
Copy link
Contributor Author

@tdykstra why is this in the wrong place in the TOC

image

Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

Well, it's in the wrong place in the toc.yml file but it's rendered in the right place and I don't know why

image

@Rick-Anderson
Copy link
Contributor Author

Rick-Anderson commented Nov 19, 2024

Well, it's in the wrong place in the toc.yml file but it's rendered in the right place and I don't know why

Not only that, it's not taking the name I specified

          - name: Configure OpenID Connect 
            uid: security/authentication/configure-oidc-web-authentication

It's using Configure OIDC web authentication
that string is nowhere to be found in the doc or the toc.yml, it does have that string in the uid:
configure-oidc-web-authentication

@tdykstra
Copy link
Contributor

It almost has to be a cache issue, but I've tried many hard refreshes, and nothing changes.

@Rick-Anderson
Copy link
Contributor Author

It almost has to be a cache issue, but I've tried many hard refreshes, and nothing changes.

Cached on the server not the client. I've tried different browsers. Look at the https://github.com/dotnet/AspNetCore.Docs/pull/34171/files and compare to the TOC

I'm going to clean up the toc.yml, merge, and see what happens.

@Rick-Anderson Rick-Anderson enabled auto-merge (squash) November 19, 2024 00:42
@Rick-Anderson Rick-Anderson merged commit fc08479 into main Nov 19, 2024
3 checks passed
@Rick-Anderson Rick-Anderson deleted the tocf branch November 19, 2024 00:43
@tdykstra
Copy link
Contributor

The file and the rendered page are now in sync -- but both are in the wrong place. 😄

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.

Hook up Configure OpenID Connect Web to TOC

3 participants