Skip to content

Option: TeSS acknowledgments#1124

Merged
fbacall merged 2 commits into
ElixirTeSS:masterfrom
kennethrioja:acknowledgement
Oct 6, 2025
Merged

Option: TeSS acknowledgments#1124
fbacall merged 2 commits into
ElixirTeSS:masterfrom
kennethrioja:acknowledgement

Conversation

@kennethrioja

Copy link
Copy Markdown
Contributor

Summary of changes

  • tess.example.yml: added about_tess_acknowledgments option, if true, displays displays ELIXIR TeSS acknowledgments in the bottom /about
  • en.yml: added the relevant texts
  • amendedtess.html.erb and created _about_tess_acknowledgments.html.erb to display it
  • Added the relevant tests in about_controller_test.rb

Motivation and context

Good idea from https://taxila.nl/about#link_tess, doing it thanks to them. IMO it should be mandatory for any other TeSS instance

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.

@kennethrioja
kennethrioja deleted the acknowledgement branch July 15, 2025 15:16
@kennethrioja
kennethrioja restored the acknowledgement branch July 15, 2025 15:16
@fbacall fbacall reopened this Sep 24, 2025

@fbacall fbacall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It might fit better if this was in the "Acknowledgements" section, what do you think?

Comment thread config/locales/en.yml Outdated
discover_learning_paths: Discover Learning Paths
discover_workflows: Discover Workflows
tess: >
TeSS is based on the <a href='https://elixir-europe.org/' target='_blank'>ELIXIR</a> TeSS architecture and could not have been realized as it is without the foundation of their work.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think change this text to something a bit milder... like:

<site name> is based on the <a href='https://elixir-europe.org/' target='_blank'>ELIXIR</a> TeSS platform.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Indeed let's keep it simple with a one liner to acknowledge ELIXIR TeSS.

@fbacall
fbacall merged commit e9394c1 into ElixirTeSS:master Oct 6, 2025
7 checks passed
@kennethrioja
kennethrioja deleted the acknowledgement branch October 7, 2025 08:49
@kennethrioja kennethrioja mentioned this pull request Oct 14, 2025
2 tasks
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