Skip to content

About link - #1365

Open
valentinRyckaert wants to merge 5 commits into
ElixirTeSS:masterfrom
valentinRyckaert:about-link
Open

About link#1365
valentinRyckaert wants to merge 5 commits into
ElixirTeSS:masterfrom
valentinRyckaert:about-link

Conversation

@valentinRyckaert

Copy link
Copy Markdown
Contributor

Summary of changes

  • Add a feature in tess.yml to add a link to the TeSS documentation
  • Add redirection from each about route to the specific documentation page (helpful for create links to the docs on other pages)

Motivation and context

Linked to this pull request

A lot of information is both in the TeSS docs and on the TeSS about page.
The goal is to merge the about page information with the TeSS documentation, and so directly redirect to it.

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree to license it to the TeSS codebase under the BSD license.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds an optional site.about_us_link configuration to redirect TeSS “About” routes to corresponding pages in the TeSS documentation, enabling instances to deprecate/replace local About-page content with external docs.

Changes:

  • Added site.about_us_link to TeSS configuration (example + test config).
  • Added AboutController before_action to redirect specific About actions to documentation URLs when configured.
  • Updated the header navigation rendering to add link options when about_us_link is present.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
test/config/test_tess.yml Adds site.about_us_link test config key (unset by default).
config/tess.example.yml Documents new site.about_us_link option and intended formatting.
app/views/layouts/_header.html.erb Adjusts nav link rendering based on about_us_link.
app/controllers/about_controller.rb Implements per-action redirects to documentation URLs when configured.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread app/views/layouts/_header.html.erb Outdated
Comment thread app/controllers/about_controller.rb
# The controller for actions related to the about pages
class AboutController < ApplicationController

before_action :check_external_link, only: [:tess, :registering, :learning_paths, :developers, :us]
valentinRyckaert and others added 2 commits August 20, 2026 14:40
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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