Skip to content

Clean up redirects.txt#12948

Merged
mik-laj merged 3 commits into
apache:masterfrom
PolideaInternal:redirects--cleanup
Dec 9, 2020
Merged

Clean up redirects.txt#12948
mik-laj merged 3 commits into
apache:masterfrom
PolideaInternal:redirects--cleanup

Conversation

@mik-laj

@mik-laj mik-laj commented Dec 9, 2020

Copy link
Copy Markdown
Member

Hello,

In Airflow 2.0, we are missing a few more files, but I will redirect them with .htaccess, because these will be moved between different documentation packages.

./executor/mesos.html
./howto/connection/aws.html
./howto/connection/gcp.html
./howto/connection/gcp_sql.html
./howto/connection/grpc.html
./howto/connection/index.html
./howto/connection/mysql.html
./howto/connection/oracle.html
./howto/connection/postgres.html
./howto/connection/ssh.html
./howto/operator/dingding.html
./howto/operator/gcp/bigtable.html
./howto/operator/gcp/cloud_build.html
./howto/operator/gcp/compute.html
./howto/operator/gcp/function.html
./howto/operator/gcp/gcs.html
./howto/operator/gcp/gcs_to_gdrive.html
./howto/operator/gcp/index.html
./howto/operator/gcp/natural_language.html
./howto/operator/gcp/spanner.html
./howto/operator/gcp/speech.html
./howto/operator/gcp/sql.html
./howto/operator/gcp/transfer.html
./howto/operator/gcp/translate-speech.html
./howto/operator/gcp/translate.html
./howto/operator/gcp/video.html
./howto/operator/gcp/vision.html
./howto/operator/papermill.html
./profiling.html

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

# New URL for Google Guide index
howto/operator/gcp/index.rst howto/operator/google/index.rst
# New URLs for Google Guides
howto/operator/gcp/ads.rst howto/operator/google/ads.rst

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.

What happens if someone navigates to /docs/stable/howto/operator/google/ads.html -- can we redirect that to the stable version of the new provider?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

In Airflow 2.0, we are missing a few more files, but I will redirect them with .htaccess, because these files are moved between different documentation packages.

I'm working on it now.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Screenshot 2020-12-09 at 14 52 18

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.

Nice one!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread docs/apache-airflow/redirects.txt Outdated
security.rst security/index.rst

# Operators guides
howto/operator/external.html howto/operator/external_task_sensor.rst

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.

did you mean it to be .rst ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point. Fixed.

@ashb

ashb commented Dec 9, 2020

Copy link
Copy Markdown
Member

@mik-laj Do you want this in before we cut 2.0.0rc1 (i.e. for 2.0.0 docs build if we do it "properly")?

@mik-laj

mik-laj commented Dec 9, 2020

Copy link
Copy Markdown
Member Author

@ashb That would be good, but we can also do it in "post production".

@ashb

ashb commented Dec 9, 2020

Copy link
Copy Markdown
Member

@mik-laj Merge this PR when you are happy with it. CI is slow, so don't wait for it.

@ashb ashb added this to the Airflow 2.0.0rc1 milestone Dec 9, 2020
@mik-laj mik-laj merged commit ec3b159 into apache:master Dec 9, 2020
@mik-laj mik-laj deleted the redirects--cleanup branch December 9, 2020 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants