search

scieloorg / opac_schema

SciELO OPAC Schema.
1 stars 10 forks source link

[Tk74] Inclusão de slug_name no modelo Pages #75

Closed robertatakenaka closed 6 years ago

robertatakenaka commented 6 years ago

Inclusão de slug_name no modelo Pages para atender https://github.com/scieloorg/opac/issues/1067

Fixes #74

jfunez commented 6 years ago

@robertatakenaka já que o campo novo slug_name é um string (livre e opcional), poderíamos ter um valor default que seria uma callable a uma função que faz o slug como é feito para o campo: title_slug aqui.

Caso seja um requisito que o usuário (admin provavelmente) mude o slug pelo admin, teríamos que validar no save para o caso que esse novo campo já tem conteúdo, que o valor no campo é um slug valido.

E seria bom também ter testes para esses ajustes 😄

jfunez commented 6 years ago

@robertatakenaka ignore meu último comentário, como vc me fez notar, seu pr já contempla o preenchimento do campo slug