nestjs/azure-storage

Add forRootAsync for configuration via configService

Aperta

#36 aperta il 24 set 2019

Vedi su GitHubGuida contributor
 (3 commenti) (2 reazioni) (1 assegnatario)TypeScript (38 fork)github user discovery
Hacktoberfesteffort1: hoursgood first issuescope: nestjs

Metriche repository

Star
 (88 stelle)
Metriche merge PR
 (Merge medio 4h 43m) (9 PR mergiate in 30 g)

Descrizione

I'm submitting a...

Current behavior

Currently, it's only possible to instantiate the module via the .withConfig method. It is not possible to inject a service into this method, so it is impossible to use a config service for the module instantiation.

Expected behavior

Enable both the .withConfig method and a factory-pattern method, that allows dependency injection.

What is the motivation / use case for changing the behavior?

I don't want to use process.env variables as all my config logic is inside a config module. It is possible to override this behavior by manually injecting all config values into process.env, but this seems like a bad choice.

Environment

Guida contributor

Direzione di ricerca
Studia i pattern forRootAsync nei moduli NestJS esistenti (es. @nestjs/typeorm), quindi implementa un metodo factory simile nel modulo Azure Storage che permetta di iniettare servizi tramite ConfigService.
Tech stack
typescriptnodejs
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
NestJSTypeScriptDependency Injection
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60