Skip to content
This repository has been archived by the owner on Nov 13, 2022. It is now read-only.

Simple coworkers view #40

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Simple coworkers view #40

wants to merge 11 commits into from

Conversation

MartinDelille
Copy link
Member

@willybrauner Voici une implementation basique d'une page affichant tous les coworkers.


{% for p in page.collection %}
<h2>{{ p.header.custom.first_name}} {{p.header.custom.last_name }}</h2>
<img src="coworkers/{{ p.title | lower }}/{{ p.header.custom.profile_picture }}" alt="{{p.header.custom.first_name}} {{p.header.custom.last_name}}"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MartinDelille Ici c'est risqué de prendre p.title car il pourrait y avoir une différence entre le nom de la page et le slug url dont on a besoin ici qui est le nom du dossier.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est réglé!

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

Successfully merging this pull request may close these issues.

None yet

2 participants