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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

serhiisamko091184
Copy link
Contributor

See the issue:
#604

@pgorod
Copy link
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
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