Author Merge Interface is highly labor intensive

Ouverte Adaptée aux débutants
#11,852 6 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
75/100
Type d'issue
Fonctionnalité
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
javascript, python
Domaine
frontend

Piste de recherche

Commencez par lire openlibrary/templates/merge/authors.html et openlibrary/plugins/openlibrary/js/merge.js afin de retracer la manière dont les entrées d’auteurs sont sélectionnées lors de la soumission du merge. Vérifiez que les entrées sont sélectionnées initialement, qu’elles peuvent être désélectionnées via la zone cliquable plus grande et que le workflow reste utilisable pour la vérification par les bibliothécaires.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Affects: Librarians Lead: @cdrini Needs: Breakdown Type: Feature Request
Feature Request

Problem / Opportunity

The interface for submitting an author merge once a set of authors has been selected in the ILE requires too much work. Even though the user has already selected a set of authors, the merge page requires you to manually select them all again, aiming for a very small click target on each click.

Super librarians approving author merges have to manually click each entry in order to approve a submitted request, which can be taxing when author merges include many entries (as merges of popular authors often do).

By far the majority of author merge submissions require no changes and are approved as submitted, so this process creates a lot of extra work.

Proposal

I am submitting a PR which does two things:

  1. It flips the workflow so that all author entries are selected by default, and the librarian can review them and deselect any that aren't appropriate for the merge.

Since only Super Librarians can finalize an author merge, the risk is low, and the elimination of wasted effort is a big win.

  1. It increases the size of the click target for selecting/deselecting entries so that users can focus on the actual task rather than spending effort trying to click in the exact right place.
Breakdown
Implementation Details (for maintainers)

This requires a few small changes to set all author entries as checked by default, move the checkbox label to include more of the content, and update the associated JS file to handle the changes.

Related files
  • openlibrary/templates/merge/authors.html
  • openlibrary/plugins/openlibrary/js/merge.js
Stakeholders

@mheiman @onnotasler

Instructions for Contributors
Langage dominant
Python
Étoiles
6.7k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
121

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.

Autres issues de internetarchive/openlibrary

Toutes les issues de internetarchive/openlibrary

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.