internetarchive/openlibrary

Publisher with / in there name are not shown

Aberta

#2.202 aberto em 29 de jun. de 2019

Ver no GitHubGuia do colaborador
 (10 comentários) (0 reação) (0 responsável)Python (1.962 forks)batch import
Lead: @cdriniPriority: 3Theme: PublishersType: Bughacktoberfest

Métricas do repositório

Stars
 (6.624 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Description

Looking for the right name for the publisher of a book I added, I found that there were already a few with slightly different names:

  • Editions 10/18 548 books
  • 10-18 177 books
  • 10/18 85 books
  • Éditions 10/18 21 books

When clicking on "10-18" you can see the page of the publisher, but clicking on any other (say "10/18") the "/" in the name is kept as is in the URL, making it a sub-directory of a nonexistent publisher.

Evidence / Screenshot (if possible)

Relevant url?

https://openlibrary.org/search/publishers?q=10%2F18&search=&m=edit&has_fulltext=true

https://openlibrary.org/publishers/10/18 compared to https://openlibrary.org/publishers/10-18

Expectation

I expected to see the publisher info when clicking on the link.

Details

  • Logged in (Y/N)?
  • Browser type/version?
  • Operating system?

Proposal & Constraints

Maybe escape the "/" in the URL somehow?

Stakeholders

Thanks!

Guia do colaborador

Direção de pesquisa
Investigue como o OpenLibrary lida com URLs de editoras. Olhe para a configuração de roteamento e a codificação de URL para caracteres especiais como '/'. Considere codificar a barra na URL ou usar uma abordagem diferente, como um slug.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65