Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fsimonis
Copy link
Member

This adds redirects for the pages /adapter-X for X in { calculix, deal.ii, su2 } adapters to their overview pages.

I also added common synonyms deal.ii and ccx.

Closes #79

@fsimonis fsimonis self-assigned this Mar 14, 2022
@fsimonis fsimonis requested a review from MakisH March 14, 2022 13:54
@MakisH
Copy link
Member

MakisH commented Mar 22, 2022

Nice, thanks!

Two points:

  • Can we drop the file extension (.html)? I guess a redirect is better, the shorter it is.
  • I would not have multiple redirects, because then we lose track. I think we should have only one per page and use it everywhere. In that context, I would go with the full name, to not have to guess every time.

@fsimonis
Copy link
Member Author

Can we drop the file extension (.html)? I guess a redirect is better, the shorter it is.

The client-side redirection is a html file, but you can omit the html anyhow. (Something we should probably do with all internal links)

I would not have multiple redirects, because then we lose track.

👍 🏗️

@fsimonis fsimonis merged commit 18c0339 into master Mar 23, 2022
@fsimonis fsimonis deleted the add-adapter-redirects branch March 23, 2022 14:55
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.

Reserved URLs for adapter pages

2 participants