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

[Website] Copy .htaccess to deploy site #506

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Apr 25, 2024

Follow on to #505 to actually copy the .htaccess file to the deployment site. Thanks to @kou for noticing this #505 (comment)

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

Let's try this! (It's better that we use a Jekyll feature instead of manual copy but this will work too.)

@kou kou merged commit 346de0b into apache:main Apr 25, 2024
1 check passed
@kou kou changed the title [Website]: Copy .htaccess to deploy site [Website] Copy .htaccess to deploy site Apr 25, 2024
@alamb
Copy link
Contributor Author

alamb commented Apr 25, 2024

Thanks @kou -- the htaccess file is now present on asf-site https://github.com/apache/arrow-site/blob/asf-site/.htaccess 🎉

I also tested a few links like
https://arrow.apache.org/datafusion/library-user-guide/working-with-exprs.html
https://arrow.apache.org/datafusion/user-guide/cli/index.html

And they seem to have redirected successfully to the new site https://datafusion.apache.org/user-guide/cli/index.html

Thank you again @kou for all your help 🙏

@alamb alamb deleted the alamb/copy_htaccess branch April 25, 2024 00:43
alamb added a commit that referenced this pull request Apr 25, 2024
Closes apache/datafusion#10124

# Rationale
See apache/datafusion#10124 -- basically we
moved a URL that is bookmarked / comes up in google search. We need to
redirect to the new location

# Changes
Use the `.htaccess` process added in
#505
#506 with @kou to redirect to
the new location
@kou
Copy link
Member

kou commented Apr 25, 2024

Great!

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