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

[fr] Move new observability-primer page under docs #5913

Open
chalin opened this issue Jan 10, 2025 · 1 comment
Open

[fr] Move new observability-primer page under docs #5913

chalin opened this issue Jan 10, 2025 · 1 comment
Labels
bug Something isn't working docs lang:fr

Comments

@chalin
Copy link
Contributor

chalin commented Jan 10, 2025

Originally posted by @chalin in #5833 (comment):

@iguitton - ah, it's because the newly added page isn't under docs. The page was published at:

You'll need to move it to be under content/fr/docs, i.e., https://opentelemetry.io/fr/docs/concepts/observability-primer/.

@chalin
Copy link
Contributor Author

chalin commented Jan 10, 2025

@open-telemetry/docs-maintainers - for the record, we could have detected this via:

$ npm run check:i18n | grep fr/
File not found  content/en/concepts/observability-primer.md - content/fr/concepts/observability-primer.md - en was removed or renamed

Should we fail a build check when locale pages are no longer found? Or maybe have a PR check that would add a comment to that effect? I'd say not at this point, but it might be worth considering.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docs lang:fr
Projects
Status: No status
Development

No branches or pull requests

1 participant