ipfs/ipfs-desktop

Multiple "configuration" files

Fermée

#1 082 ouverte le 4 sept. 2019

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 personne assignée)JavaScript (912 forks)batch import
P3help wanted

Métriques du dépôt

Stars
 (5 632 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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!

Guide contributeur

Direction de recherche
La configuration actuelle est stockée à deux emplacements : un dans le répertoire de configuration de l'application (voir le lien dans l'issue) et ~/.ipfs desktop. Examinez le code qui lit et écrit ces fichiers, en particulier config.json et le fichier IPFS PATH. Envisagez de déplacer toute la configuration vers un seul emplacement, de mettre à jour les références et de tester cross platform.
Stack technique
javascript
Domaine
desktop
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65