Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajoute le support pour les billets multi-auteurs #32

Closed
wants to merge 8 commits into from

Conversation

RLesur
Copy link
Member

@RLesur RLesur commented Mar 26, 2020

Voici une proposition pour #30.
note: j'ai rajouté dans #30 un exemple de bug qu'il y a dans la version actuelle du site.

Pour générer autant de liens qu'il y a d'auteurs, le champ author doit être un Array au lieu d'un string.
C'est la raison pour laquelle je n'ai pas proposé de PR modifiant le thème airspace. En effet, cela briserait tous les sites utilisant airspace.

Plus précisément, c'est parce que je n'ai pas trouvé de moyen de supporter à la fois un string et un Array (mais je ne suis pas un expert d'Hugo). Voir par exemple cette discussion sur ce sujet : https://discourse.gohugo.io/t/how-to-check-if-a-variable-is-string-or-array/7460

Si on arrivait à trouver le moyen de tester si le champ author est un Array ou un string alors on pourrait proposer une PR à airspace.

En attendant, le seul moyen que j'ai trouvé est de passer tous les champs author en Array et de boucler dessus. Ca fonctionne bien.

Inconvénient de cette PR : cela impose de modifier plusieurs partials.

Qu'en pensez vous ?


fix #30

@RLesur RLesur changed the title Issue 30 Ajoute le support pour les billets multi-auteurs Mar 26, 2020
@strainel
Copy link
Contributor

Cela me parait une super idée. Et très élégant.
Je viens de faire une issue tout de même pour tester leur "avis" sur une éventuelle PR à ce sujet.
themefisher/airspace-hugo#71

@strainel
Copy link
Contributor

@RLesur : tu mets à jour ta pr ?

@RLesur
Copy link
Member Author

RLesur commented Mar 31, 2020

@strainel j'en ferai plutôt une autre, je trouve ça plus propre. Je ferme celle là.

@RLesur RLesur closed this Mar 31, 2020
@RLesur RLesur deleted the issue-30 branch March 31, 2020 19:20
@RLesur
Copy link
Member Author

RLesur commented Mar 31, 2020

FTR, cette PR a été remplacée par #62.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auteurs multiples
2 participants