-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Subdomains #17
base: main
Are you sure you want to change the base?
Subdomains #17
Conversation
Required to support subdomain
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/redd/howurls-work/jhhdizgk4 |
Hey, @niklas-e! Thank you so much for this contribution! I'll give it a closer look sometime next week. I think we should introduce a fallback to English in case there's no translation for strings. I can do this as a separate pull request that precedes this one. |
Yeah, fallback would be great. Didn't actually notice there wasn't one already as I didn't check the localization implementation. I guess I'm too used to have react-i18next or something similar in my projects. |
I take that back. Following the spirit of previous panels, including the separator is intuitive enough. |
I just noticed this is still open. Is there some prerequisites before this can be merged @kettanaito?
Have you implemented this? |
@niklas-e, no, I haven't got the chance to implement the language fallback. It should be rather straightforward to do. Right now, if the translation is missing, it will show the label IDs: This isn't nice. I can help translating the new section into as many languages I know but looks like we need some help from the community to provide other translations (such as Chinese, for example). If you are final with the English description, I can post this pull request around and I'm sure people will help with translations! |
I wonder if the dot following the subdomain is considered a part of the subdomain? 🤔 Right now, the way it's highlighted in the UI implies that is is. |
Resolves #1
Added section for subdomains. I tried to keep the description in line with others, but if you have any suggestions or ideas, I can make changes 🙂