Fix contributor guide URL

[Metal-Bat]

fixes #4971

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: Metal-Bat / name: Erfan (91ac530, 3b9d995)

[chalin]

Thanks for the fix @Metal-Bat.

LGTM, but ... I have a question for @open-telemetry/docs-maintainers: generally, an en page fix must not be propagated to non-en languages, since the locale maintainers need to update the default_lang_commit too. We might want to make an exception when a 404 is being fixed? WDYT?

I'm leaning towards this PR be exclusively for en pages. Other locales can handle their own page updates. But I'm willing to be convinced otherwise. I don't have a strong opinion about this.

/cc @emdneto

[Metal-Bat]

I think you are making a good point
so if teams accept this idea i would be glad to add persian documents to this project

Thank you everyone

[theletterf]

I'm leaning towards this PR be exclusively for en pages.

+1. Exceptions might become unmanageable in the long run.

[svrnm]

I'm leaning towards this PR be exclusively for en pages.

+1. Exceptions might become unmanageable in the long run.

Agred.

so if teams accept this idea i would be glad to add persian documents to this project

Thanks for the offer @Metal-Bat! Can you raise a dedicated issue for that? Note, that we need at least 2 people, 3 would be even better to work on that, since we as non-persian speakers can not help you with reviewing the content. Also note, that localizations are still a new thing to the OpenTelemetry website, so onboarding of new languages is still a slow process

[chalin]

@Metal-Bat - please drop the non-en page changes from this PR.

[Metal-Bat]

I'm leaning towards this PR be exclusively for en pages.

+1. Exceptions might become unmanageable in the long run.

Agred.

so if teams accept this idea i would be glad to add persian documents to this project

Thanks for the offer @Metal-Bat! Can you raise a dedicated issue for that? Note, that we need at least 2 people, 3 would be even better to work on that, since we as non-persian speakers can not help you with reviewing the content. Also note, that localizations are still a new thing to the OpenTelemetry website, so onboarding of new languages is still a slow process

sure i will open an issue and i will find some people to start this documentation

[Metal-Bat]

I'm leaning towards this PR be exclusively for en pages.

sure

[chalin]

✨

[chalin]

Thanks all!

sure i will open an issue and i will find some people to start this documentation

@Metal-Bat et al. - heads up that Docsy (and hence this repo) doesn't fully support RTL languages yet. I'm working on it, but the feature might not land for a few weeks or months -- though I do see it happening before the end of the year. Reference: google/docsy#1442.

[Metal-Bat]

Thanks all!

sure i will open an issue and i will find some people to start this documentation

@Metal-Bat et al. - heads up that Docsy (and hence this repo) doesn't fully support RTL languages yet. I'm working on it, but the feature might not land for a few weeks or months -- though I do see it happening before the end of the year. Reference: google/docsy#1442.

great thanks