loopbackio / loopbackio/loopback-next

loopback-datasource-juggler datasource definition for setup method is missing

Ouverte
#7,115 4 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@achrinza y travaille déjà.

Depuis le 5/10/2021.

bug
Langage dominant
TypeScript
Étoiles
5.1k
Forks
1.1k
Merge moyen
2 j 21 h
PR mergées (30 j)
27

Description

Steps to reproduce

At https://github.com/strongloop/loopback-datasource-juggler/blob/master/types/datasource.d.ts,
there is no definition for method setup, which is defined at:
https://github.com/strongloop/loopback-datasource-juggler/blob/06428247ad1cc48bce82d135f795fb3eaae28b33/lib/datasource.js#L382

  1. I create custom datasource class MyDataSource which extends juggler.Datasource.
  2. I create method setup inside class MyDataSource .
  3. I can not use: super.setup() inside method setup.

Current Behavior

When I want to use super.setup(), there will be ts error: Property 'setup' does not exists on type 'Datasource'.
Selection_161

Expected Behavior

No ts error when I want to use super.setup() inside setup method on my custom datasource class.

Additional information

Setup method definition:

  /**
   * Setup datasource
   * @param dsName The name of the datasource.
   * @param settings The datasource settings
   */
  setup(dsName: string, settings: Options): void;
  setup(settings: Options): void;

Guide de contribution

Ouvrir le guide de contribution

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.

É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.