all-contributors / all-contributors/cli

[BUG]: Markdown provided in `types[key].link` is injected as-is into HTMLAnchorElement's href attribute

Ouverte
#520 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
425
Forks
149
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### 📝 Describe the bug

When using the recommended setup when defining custom types, the following configuration:

```json
"types": {
"core": {
"symbol": "🏵️",
"description": "Core team",
"link": "[<%= symbol %>](https://github.com/bl-portal-bot)"
}
}
```

...ends up generating the following cell:

```html


Terry Mun-Andersen


Terry Mun-Andersen





🏵️

💻
```

...which is incorrect, as the markdown is injected as-is into the `href` attribute of the HTMLAnchorElement. Updating the `link` field to just contain the URL will then generate the correct output:

```json
"types": {
"core": {
"symbol": "🏵️",
"description": "Core team",
"link": "https://github.com/bl-portal-bot"
}
}
```

```html


Terry Mun-Andersen


Terry Mun-Andersen





🏵️

💻

```

### 👣 Steps to Reproduce

Use the following configuration to add a custom emoji key as [described in the docs](https://allcontributors.org/en/bot/configuration/):

```json
"types": {
"core": {
"symbol": "🏵️",
"description": "Core team",
"link": "[<%= symbol %>](https://github.com/bl-portal-bot)"
}
}
```

This results in the markdown string being inserted as-is into the `href` attribute.

### 🎯 Expected behavior

Using `"link": "[<%= symbol %>](https://github.com/bl-portal-bot)"` should generate the entire HTMLAnchorElement correctly.

### 💡 Additional context

_No response_

### 🤝 Participation

- [ ] I am willing to submit a pull request for this issue.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.