@bem-react/core Proposal withBemMod with type check for mod

Ouverte
#458 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

@belozer y travaille déjà.

Depuis le 9/8/2019.

  • #463 par @belozer — ouverte

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
react, typescript
Domaine
frontend

Piste de recherche

Commencez par le point d’entrée withBemMod et examinez ses définitions TypeScript ainsi que la manière dont l’objet mod est typé. Confirmez le comportement actuel avec l’exemple, puis déterminez l’erreur de type attendue et la complétion de l’éditeur pour une valeur mod invalide. C’est terminé lorsque l’argument mod rejette les valeurs qui ne font pas partie des props du composant et fournit des suggestions tenant compte des types.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

export interface IconFaInstagramProps {
    fa?: 'instagram'
}

export const withFaInstagram = withBemMod<IconFaInstagramProps>(cnIcon(), {
    fa: 'Instagram' // -> TS not work
}, Icon => props => (<Icon {...props}><FaInstagram /></Icon>));

На текущий момент нет как типизации так и подсказок при наборе.
Приходится тратить на это время и внимание (хоть мы и прокинули интерфейс уже...).

Langage dominant
TypeScript
Étoiles
450
Forks
69
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de bem/bem-react

Toutes les issues de bem/bem-react

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

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