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.

Sign up for GitHub

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

Jump to bottom

chore(redirects): add redirects for Modeler #510

Merged
merged 1 commit into from
Nov 6, 2024
Merged

chore(redirects): add redirects for Modeler #510

merged 1 commit into from
Nov 6, 2024

Conversation

mboskamp
Copy link
Member

@mboskamp mboskamp commented Nov 6, 2024

@mboskamp mboskamp self-assigned this Nov 6, 2024
@mboskamp mboskamp mentioned this pull request Nov 6, 2024
Closed
@mboskamp mboskamp requested a review from psavidis November 6, 2024 10:02
@psavidis
Copy link
Contributor

psavidis commented Nov 6, 2024 •
edited
Loading

Review Notes

  • Compared the two .htaccess files (stage, live); the content is identical (as it should be) for the rules:
    • RewriteRule Modeler Latest
    • RewriteRule Modeler Develop
    • RewriteRule Modeler 7.17
  • The added rules are added in this PR to the .htaccess files
  • The removed rules are absent from the .htaccess files

@mboskamp mboskamp merged commit 70b0a61 into master Nov 6, 2024
1 check passed
@mboskamp mboskamp deleted the 4765-centralize-redirects branch November 6, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psavidis psavidis psavidis approved these changes

Assignees

@mboskamp mboskamp

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mboskamp @psavidis