Holo-Host/holo-auth

Fix HoloPort URL in challenge email

Closed this issue · 3 comments

cc @alastairong @jarodholtz

I noticed that the following part of challenge template:

<a href="https://{{holochain_agent_id}}.holohost.net">{{holochain_agent_id}}.holohost.net</a>

Has been changed to:

<p>https://{{holochain_agent_id}}.holohost.net</p>

I'm not sure what the motivation is. I'd guess that it was made into text rather than a link because we previously had "Link not working?" text there, and that no longer applies, it's a completely different link to the one that is used as part of the button.

Additionally, I optimized the template so that the entire address would fit, and because https:// is now visible part of this line, it no longer fits:

Screenshot from 2020-01-30 09-19-49

Unless anyone minds otherwise, can I revert this back? If no, I'll just close this because this is way too minor to have an argument over.

cc @yegortimoshenko

The motivation to make it not a link is to reduce the chance of someone clicking it instead of the button and thereby trying to access HP Admin before registration has completed.

If this will automatically redirect to hpstatus then maybe it's not a problem, but I would suggest we just close it and I will mention to Jarod to adjust the template width accordingly.

If this will automatically redirect to hpstatus then maybe it's not a problem, but I would suggest we just close it and I will mention to Jarod to adjust the template width accordingly.

Aha, I see. I'll tweak the template width then.

Bumped template width to 640, this is not an issue anymore. Thank you for explaining me the rationale. Sorry for jumping to conclusions.