HelloZeroNet / HelloZeroNet/ZeroNet

Calling mergerSiteDelete doesn't do anything

Ouverte
#1,238 1 commentaire 1 réaction 1 personne assignée Voir sur GitHub

@HelloZeroNet y travaille déjà.

Depuis le 17/1/2018.

bug
Langage dominant
JavaScript
Étoiles
18.8k
Forks
2.3k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Step 1: Please describe your environment
  • ZeroNet version: 0.6.0
Step 2: Describe the problem:

Calling my removeMerger function here, I get the following output:

Removing 15UgXhtcVqf6KTEBVJBkwt5SyKxgcmDDcZ

/* Before */
{…}
/* ... */
14A1qSiNVjcPj1rqBNd49JFJWfHgDvePVU
15UgXhtcVqf6KTEBVJBkwt5SyKxgcmDDcZ // bye bye!
15cwpzZmWmYyGmLFu22FDMKWAfSeAuWbqp
17E4GPZqVnSQSVQEzjiG2JCeLLVwjLvYpR
17izouNc83HLAxASAJfJMr9XW2oLUCH3dH

// mergerSiteDelete called

/* After */
{…}
/* ... */
14A1qSiNVjcPj1rqBNd49JFJWfHgDvePVU
15UgXhtcVqf6KTEBVJBkwt5SyKxgcmDDcZ // Oh, still there? Hrm...
15cwpzZmWmYyGmLFu22FDMKWAfSeAuWbqp
17E4GPZqVnSQSVQEzjiG2JCeLLVwjLvYpR
17izouNc83HLAxASAJfJMr9XW2oLUCH3dH
Observed Results:

After calling mergerSiteDelete, and mergerSiteList before and afterwards, the merger site isn't actually deleted, neither from the list, nor the client. It still shows up on ZeroHello.

Expected Results:

Ideally the site would be removed from the mergerSiteList and site would be deleted from the client, removing it from ZeroHello. I believe this is what the documentation implies.

Test it live on the 'genres' tab of ZeroLSTN. Edit: Removed due to confusion until this bug is fixed.

Thanks!

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

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.