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

Subdomains #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Subdomains #17

wants to merge 2 commits into from

Conversation

niklas-e
Copy link
Contributor

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 🙂

@vercel
Copy link

vercel bot commented May 16, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/redd/howurls-work/jhhdizgk4
✅ Preview: https://howurls-work-git-fork-niklas-e-subdomains.redd.now.sh

@kettanaito
Copy link
Member

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.

@niklas-e
Copy link
Contributor Author

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.

@d-damien
Copy link
Contributor

d-damien commented Apr 22, 2021

Technically the dot isn't part of the subdomain. We need to tell domains, sub- and TLDs, are separated by dots. I think an additional panel is needed ; hopefully it won't be too hard to see & click on a dot.

I take that back. Following the spirit of previous panels, including the separator is intuitive enough.

@niklas-e
Copy link
Contributor Author

I just noticed this is still open. Is there some prerequisites before this can be merged @kettanaito?

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.

Have you implemented this?

@kettanaito
Copy link
Member

kettanaito commented Dec 11, 2023

@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:

Screenshot 2023-12-11 at 19 03 56

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!

@kettanaito
Copy link
Member

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.

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.

Sub-domains
3 participants