Skip to content

Conversation

@ischoegl
Copy link
Member

@ischoegl ischoegl commented Aug 23, 2025

@ischoegl ischoegl requested a review from a team August 24, 2025 02:10
@ischoegl
Copy link
Member Author

Should be merged after Cantera/cantera-example-data#5 and Cantera/cantera#1951.

@ischoegl
Copy link
Member Author

I believe this is ready to be reviewed/merged.

Copy link
Member

@speth speth left a comment

Choose a reason for hiding this comment

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

In addition to the present set of rules, which resolve issues with external links once 3.2 becomes stable, there some additional rules to add that will allow the version switcher to work when you're on one of the corresponding pages, since it just looks for the same URL with the version path element changed. I'll push a commit with what I think would work for these rules.

Comment on lines 119 to +120
RewriteRule "^examples/jupyter/reactors/interactive_path_diagram.ipynb.html" "stable/examples/python/kinetics/interactive_path_diagram.html" [R=permanent,L]
RewriteRule "^examples/jupyter/reactors/nonideal_shock_tube.ipynb.html" "stable/examples/python/reactors/NonIdealShockTube.html" [R=permanent,L]
RewriteRule "^examples/jupyter/reactors/nonideal_shock_tube.ipynb.html" "stable/examples/python/reactors/non_ideal_shock_tube.html" [R=permanent,L]
Copy link
Member

Choose a reason for hiding this comment

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

I guess we still have website updates that can't be merged until the next release. "stable" currently points to the 3.1 docs, where these have the IntialCaps names.

RewriteRule "^examples/jupyter.*" "stable/examples/python/index.html" [R=permanent,L]

# Redirect from CamelCase to snake_case examples
RewriteRule "^stable/examples/python/reactors/NonIdealShockTube.html$" "stable/examples/python/reactors/non_ideal_shock_tube.html" [R=permanent,L]
Copy link
Member

Choose a reason for hiding this comment

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

Likewise these redirects should only be active after the Cantera 3.2 release.

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.

Add StanShock to affiliated projects

2 participants