Fixed email link issue#16444

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

TimoBakx wants to merge1 commit intosymfony:4.4fromTimoBakx:patch-1

Closed

Fixed email link issue#16444

TimoBakx wants to merge1 commit intosymfony:4.4fromTimoBakx:patch-1

Conversation

Copy link

Member

TimoBakx commentedJan 24, 2022

Formatting (bold) breaks the mailto-link.

This ended up as:

<a href="mailto:**care@symfony.com">**care@symfony.com</a>**:

carsonbot added the Status: Needs Review label

Jan 24, 2022

carsonbot added this to the4.4 milestone

Jan 24, 2022

BahmanMD suggested changes

Jan 25, 2022

View reviewed changes

contributing/code_of_conduct/care_team.rst OutdatedShow resolvedHide resolved

carsonbot added Status: Needs Work and removed Status: Needs Review labels

Jan 25, 2022

Fixed email link issue

d0d6a73

Formatting (bold) breaks the mailto-link when used without spacing.

carsonbot added Status: Needs Review and removed Status: Needs Work labels

Jan 25, 2022

Copy link

Member

wouterj commentedJan 25, 2022

Thank you Timo! This seems like a bug in the PHP rst parser we're using, but I think this workaround is better than trying to fix the bug for now :)

wouterj added a commit that referenced this pull request

Jan 25, 2022

minor#16444Fixed email link issue (TimoBakx)

f2b24d0

This PR was merged into the 4.4 branch.Discussion----------Fixed email link issueFormatting (bold) breaks the mailto-link.This ended up as:```<a href="mailto:**care@symfony.com">**care@symfony.com</a>**:```Commits-------7df7773 Fixed email link issue

Copy link

Member