EBISPOT / OLS

Ontology Lookup Service from SPOT at EBI
http://www.ebi.ac.uk/ols
Apache License 2.0
96 stars 40 forks source link

implemented rebrandable OLS with environment variables for #438 #482

Closed giraygi closed 2 years ago

giraygi commented 3 years ago

Current debranding feature disables the use of some functionalities. For this reason, I have introduced a new variable to be able to hide some EBI specific content. Plus, I have introduced several branding variables for adapting some page content to the identity of another institution willing to maintain the service. These variables can be assigned to the docker-compose file as environment variables.

matentzn commented 3 years ago

Super cool @giraygi, do you have a public instance somewhere where I can see how your changes look like life? I am not directly on the OLS team, but interested in your changes for some of our custom OLS projects.

giraygi commented 3 years ago

Super cool @giraygi, do you have a public instance somewhere where I can see how your changes look like life? I am not directly on the OLS team, but interested in your changes for some of our custom OLS projects.

Thanks a lot @matentzn . We have two public instances to give you an idea about my changes. They are: TS of NFDI4Ing and TS of TIB Central. The footer is not very fancy yet since I wanted to stick to the code of EBI OLS as much as possible but I will update it soon once my changes are approved. Plus, this rebranding currently excludes favicons. I will also add it later.

matentzn commented 3 years ago

This is really amazing! :) Nice job!

jamesamcl commented 3 years ago

This is great, thank you! I will get it merged this week. 😃

giraygi commented 3 years ago

Thanks a lot guys! I am very glad that I might be of any help :)

giraygi commented 2 years ago

@udp Hello again. I wanted to remind you that I had made a few corrections (4 more commits on 2nd of June ) after you confirmed my pull request.

jamesamcl commented 2 years ago

Thanks!

giraygi commented 2 years ago

Thanks!

No Problem! I have a small bug-fix and documentation added for this in https://github.com/EBISPOT/OLS/pull/516