Skip to content

Add PEP 545 redirect for Persian translation - #648

Draft
sepehr-rs wants to merge 3 commits into
python:mainfrom
sepehr-rs:add-fa
Draft

Add PEP 545 redirect for Persian translation#648
sepehr-rs wants to merge 3 commits into
python:mainfrom
sepehr-rs:add-fa

Conversation

@sepehr-rs

Copy link
Copy Markdown

Description

This PR adds PEP 545 redirects for Persian translation

@sepehr-rs

Copy link
Copy Markdown
Author

@StanFromIreland You mentioned that the PR should be a draft, right? Would you mind taking a look when you get a chance?


# Python 3 docs are the default at the root of each translation.
location ~ ^/(bn-in|el|es|fr|id|it|ja|ko|pl|pt-br|ro|sv|tr|uk|zh-cn|zh-tw)/$ {
location ~ ^/(bn-in|el|es|fr|id|it|ja|ko|pl|pt-br|ro|sv|fa|tr|uk|zh-cn|zh-tw)/$ {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please keep it alphabetical and move fa before fr.

GET {{host}}/fa/
HTTP 302
[Asserts]
header "Location" == "http://localhost/fa/3/"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry, I wasn't clearer, my ordering comment applies everywhere.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Oh, no worries. I’ll apply the change globally now. Thanks for clarifying!

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.

2 participants