loopbackio / loopbackio/loopback-datasource-juggler
Strongly typed context types for operation hook typings
Ouverte
Personne n'a encore pris cette issue.
developer-experience
feature
good first issue
- Langage dominant
- JavaScript
- Étoiles
- 282
- Forks
- 366
- Merge moyen
- 17 h 4 min
- PR mergées (30 j)
- 10
Description
@bajtos suggested to provide strongly typed context types based on the hook name.
For example:
export interface AccessHookContext<T extends typeof PersistedModel> {
Model: T;
query: Filter<T>;
}
export type Listener<Ctx> = (ctx: Ctx, next: (err?: any) => void) => void;
class PersistedModel {
// ...
static observe(operation: 'access', handler: Listener<AccessHookContext>): void;
static observe(operation: 'before save', handler: Listener<BeforeSaveHookContext>): void;
// etc.
}
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par la discussion dans la pull request #1820 et les typings existants de l’operation-hook de PersistedModel.observe. Identifiez le type de contexte nécessaire pour chaque hook nommé par l’API actuelle, puis vérifiez que chaque handler reçoit le contexte correspondant et que les déclarations TypeScript restent cohérentes entre les opérations.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- backend, database
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100