Skip to content

Fix #604 - Issue about Page admin/installation-guide/Downloading#651

Open
serhiisamko091184 wants to merge 1 commit intomasterfrom
branch-006-fix-links
Open

Fix #604 - Issue about Page admin/installation-guide/Downloading#651
serhiisamko091184 wants to merge 1 commit intomasterfrom
branch-006-fix-links

Conversation

@serhiisamko091184
Copy link
Copy Markdown
Contributor

See the issue:
#604

@pgorod
Copy link
Copy Markdown
Collaborator

pgorod commented Apr 23, 2024

It's better to avoid having links such as crm.yourserver.com in the page, because they become clickable (if you remove the back-ticks), and people will click them only to go to some random server out there.

So I suggest to either put back the back-ticks, or if you prefer to remove them (which has the advantage of facilitating copying them to the clipboard), then use that <your-server> syntax everywhere (at least those links can't be clicked).

@serhiisamko091184
Copy link
Copy Markdown
Contributor Author

Hello @pgorod,

thanks for your feedback,

So I suggest to either put back the back-ticks, or if you prefer to remove them (which has the advantage of facilitating copying them to the clipboard), 
then use that <your-server> syntax everywhere (at least those links can't be clicked).

Yes, I'm planning to substitute other occurrences of in the codebase to make it consistent as well.

Thanks!

Regards,
Serhii

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants