Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make hulks website international #95

Merged
merged 5 commits into from
Apr 12, 2020
Merged

Conversation

anesssi
Copy link
Contributor

@anesssi anesssi commented Apr 9, 2020

Solves hulks/nao#2145

Description

The complete hulks.de website has been translated to English. The only exception is the Impressum (with disclaimer) which still remains German for legal reasons.
There are some small content changes on the Meet HULKs page as well: The team photo has been replaced by our "We want you" recruiting picture and the recruiting page (recruiting.hulks.de) is linked now.

Copy link
Member

@rkost rkost left a comment

Choose a reason for hiding this comment

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

Nice work with the meet page! LGTM, minor remarks - one has to be answered by @PatrickGoettsch I guess.

I also noticed that we still have a german impress and I guess that we have to keep that? Do we want to change the link to the impress on the bottom of each page to "Impress/Impressum" or should be just leave it as it is?

@@ -11,7 +11,7 @@
<h1>HULKs</h1>
<p>
<b>RoboCup SPL Team<br />
TU Hamburg, Germany</b>
TUHH, Germany</b>
Copy link
Member

Choose a reason for hiding this comment

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

I would keep TU Hamburg to keep it consistent with our current logo. Is there a reason to change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The reason why I made this change is because TU (Technische Universität) is a German expression and does not exist in the English name of the university (Hamburg University of Technology). Therefore, one could either use the English name (Hamburg University of Technology) which is fairly long, or use the official abbreviation of our university (TUHH).

Copy link
Contributor

@tuxbotix tuxbotix Apr 9, 2020

Choose a reason for hiding this comment

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

In my personal feeling, TU Hamburg or TUHH feels more "normal" than Hamburg University of Technology, in fact if you directly translate, TU Hamburg is the outcome considering pattern used by TuM, etc (Google and Deepl now show official one for TUHH). IDK why they have the official translation as it is.

meet/index.html Show resolved Hide resolved
publications/index.html Outdated Show resolved Hide resolved
sponsors/index.html Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@rkost
Copy link
Member

rkost commented Apr 9, 2020

I also noticed that we still have a german impress and I guess that we have to keep that?

Note to myself: Read the PR description first 🦄

@rkost rkost self-assigned this Apr 9, 2020
@rkost
Copy link
Member

rkost commented Apr 9, 2020

This also fixes #94

@rkost rkost linked an issue Apr 9, 2020 that may be closed by this pull request
@anesssi
Copy link
Contributor Author

anesssi commented Apr 9, 2020

I left the link to the imprint in German since the imprint is still written in German. Once we have an English version of the imprint as well, we can change it to "Imprint/Impressum".

@anesssi
Copy link
Contributor Author

anesssi commented Apr 9, 2020

I fixed the RoHOW link and the mail about thesis (in English or German) stuff.

Copy link
Member

@rkost rkost left a comment

Choose a reason for hiding this comment

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

LGTM. Finally we have an international website 👍

@rkost rkost merged commit d0a7bdf into master Apr 12, 2020
@rkost rkost deleted the makeHulksWebsiteInternational branch April 12, 2020 16:03
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.

Consider adding a link to https://hulks-recruiting.de/ to hulks.de/meet
3 participants