ipfs/ipfs-desktop

Multiple "configuration" files

Open

#1082 aperta il 4 set 2019

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (912 fork)batch import
P3help wanted

Metriche repository

Star
 (5632 star)
Metriche merge PR
 (Merge medio 8g 12h) (6 PR mergiate in 30 g)

Descrizione

Right now, IPFS Desktop is storing its configuration in two different places:

  1. Here, for both logs and config.json.
  2. ~/.ipfs-desktop for other files such as IPFS_PATH.

@lidel shared his concern on #1079 about it:

Digression: I am worried this separate file will produce bugs in the long run. Additional code is required to keep it in sync with config.json and we need to remember about it. This concern should not block this PR form being merged, just something to think about :)

We're using ~/.ipfs-desktop for things that must be easily accessible from other scripts: it's a well-known location and it works cross-platform!

The application is now small, and the burden of maintenance of this specific topic is not really bug, but might be something to think about in the future!

Guida contributor

Direzione di ricerca
La configurazione attuale è memorizzata in due posizioni: una nella directory di configurazione dell'app (vedi link nell'issue) e ~/.ipfs desktop. Rivedi il codice che legge e scrive questi file, in particolare config.json e il file IPFS PATH. Considera di spostare tutta la configurazione in un'unica posizione, aggiornare i riferimenti e testare cross platform.
Tech stack
javascript
Dominio
desktop
Tipo issue
Ricerca
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65