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

Add Doctolib Italy #7687

Open
6 of 11 tasks
freshgoods opened this issue Jul 22, 2023 · 7 comments
Open
6 of 11 tasks

Add Doctolib Italy #7687

freshgoods opened this issue Jul 22, 2023 · 7 comments
Assignees
Labels
add site Issue/PR adds a site to the repo.

Comments

@freshgoods
Copy link

Site name

Doctolib

Site URL

https://www.doctolib.it/

Documentation URL

https://doctolibpatient.zendesk.com/hc/it/articles/360021300860

Recovery Documentation URL

No response

Supported 2FA methods

  • TOTP (RFC-6238, Google Authenticator)
  • Proprietary Software (Authy, Duo, etc.)
  • U2F/WebAuthn security keys
  • Proprietary hardware (Yubico OTP, RSA keys, etc.)
  • Phone calls
  • SMS tokens
  • Email tokens

Regions served

Italy

Eligibility

Additional information

Category: Health
Customer-facing web site (different to the "professional" one)

@freshgoods freshgoods added the add site Issue/PR adds a site to the repo. label Jul 22, 2023
@kmpoppe kmpoppe self-assigned this Jul 23, 2023
@kmpoppe
Copy link
Member

kmpoppe commented Jul 23, 2023

Will be grouped together in one entry (as per additional-domains rule), assigned creating the entry to myself as I need to check for a SVG'd logo directly from DTL.

@freshgoods
Copy link
Author

Will be grouped together in one entry (as per additional-domains rule), assigned creating the entry to myself as I need to check for a SVG'd logo directly from DTL.

OK, but please bear in mind that the documentation links are unique and region-specific (in the relevant language) for each domain. I'm not sure if that's maintained if these are listed as one entity.

@hkamran80
Copy link
Member

If you remove the region path from the URL (the /it/ part), it will automatically redirect you based on your location.

@freshgoods
Copy link
Author

If you remove the region path from the URL (the /it/ part), it will automatically redirect you based on your location.

Having tried this, it appears to use your locale (not location) to set the locale code in the URL. For example, anyone with their locale set to English (or anything that isn't DE/FR/IT) will receive a 'page doesn't exist' error, which isn't helpful and just looks like a broken link. At least seeing a valid page in another language can be translated.

Maybe it will work as intended for the primary audience (those whose locale matches) but it seems this would just introduce a lot more chances of failure for everyone else.

@kmpoppe
Copy link
Member

kmpoppe commented Jul 24, 2023

When I use the /en/ link, having my locale set to German, it redirects me to /de/. If hc redirects you to a language that doesn't exist even if the link is to an existing language, that's bad UX behaviour, but nothing we can change.
Let's link to /en/..., that'll work for the target audience and a significant part of the rest of the world I would guess.

@freshgoods
Copy link
Author

freshgoods commented Jul 24, 2023

Let's link to /en/..., that'll work for the target audience and a significant part of the rest of the world I would guess.

If we must only have one entry and only one documentation link, please use one of the existing ones (/de/ or /fr/), because /en/ isn't a valid link and results in 'page doesn't exist'.

Or do we have to use an invalid locale to have it redirect automatically to the locales that do match?

@kmpoppe
Copy link
Member

kmpoppe commented Jul 25, 2023

If we must only have one entry and only one documentation link, please use one of the existing ones (/de/ or /fr/), because /en/ isn't a valid link and results in 'page doesn't exist'.

I shall be trying multiple ways on how this can work before adding the entry to the site. What is your location, operating system, browser and locale so that I can try your environment on those entries?

K

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add site Issue/PR adds a site to the repo.
Projects
None yet
Development

No branches or pull requests

3 participants