Skip to content

Comments

chore: changes to use https, and redirect of some capital letters in url#1756

Closed
Meng-Heng wants to merge 1 commit intomasterfrom
404-and-dupes-urls
Closed

chore: changes to use https, and redirect of some capital letters in url#1756
Meng-Heng wants to merge 1 commit intomasterfrom
404-and-dupes-urls

Conversation

@Meng-Heng
Copy link
Collaborator

More on: keymanapp/keyman.com#415.

The .htaccess validation may be overlapping a bit with the existing rules in the file but this should remove the registration of 404 Not found URLs:

https://help.keyman.com/DEVELOPER/ENGINE/web/1.0/
...
https://help.keyman.com/DEVELOPER/ENGINE/web/1.0/index.php
...
https://help.keyman.com/DEVELOPER/ENGINE/WEB/10.0/guide/examples/automatic-control
https://help.keyman.com/DEVELOPER/ENGINE/web/10.0/guide/examples/automatic-control
...
https://help.keyman.com/DEVELOPER/ENGINE/web/11.0/reference/core/addHotKey.php
...
https://help.keyman.com/DEVELOPER/ENGINE/web/12.0/guide/examples
...
https://help.keyman.com/DEVELOPER/ENGINE/WEB/13.0
https://help.keyman.com/DEVELOPER/ENGINE/web/13.0/guide/
...
https://help.keyman.com/DEVELOPER/ENGINE/web/14.0/reference/core/
https://help.keyman.com/DEVELOPER/ENGINE/WEB/14.0/reference/core/addKeyboards
...=
https://help.keyman.com/DEVELOPER/ENGINE/WEB/15.0/guide/examples
...
https://help.keyman.com/DEVELOPER/ENGINE/web/16.0/
...
more

Descriptive reports: https://docs.google.com/spreadsheets/d/1On3WrJMyDo3QboQvl7S1sbtD1z3HO3JH/edit?usp=sharing&ouid=115939244027961768677&rtpof=true&sd=true

@Meng-Heng Meng-Heng added this to the A18S17 milestone Dec 18, 2024
Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

Can you separate the .htaccess change (redirect of some capital letters in url) to a separate PR?

From #691, it sounded more complicated to check all the scenarios,

@Meng-Heng
Copy link
Collaborator Author

Yes, that sounds good @darcywong00.

#1758 + #1759

@Meng-Heng Meng-Heng closed this Dec 19, 2024
@mcdurdin mcdurdin deleted the 404-and-dupes-urls branch January 13, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants