search

DIAGNijmegen / website-content

This repository stores all the content for the diag websites.
MIT License
6 stars 76 forks source link

Add multiple positions to DIAG member profile #285

Closed njmhendrix closed 3 years ago

njmhendrix commented 4 years ago

Describe the bug Besides being an associate professor, Matthieu Rutten is a radiologist as well. He wants to add multiple positions to his profile on the DIAG website. Thus, I added "radiologist" to his profile (content/pages/members/matthieu-rutten.md) using this line "position: Associate Professor, Radiologist" (see commit #284). This text is properly displayed on the people page (https://www.diagnijmegen.nl/people/), but not on a vacancy page (https://www.ai-for-health.nl/vacancies/fracture-detection-radius-ulna-metacarpals/). On the latter page, the text is displayed as a list (see screenshots).

To Reproduce To reproduce the behaviour, add multiple positions to the "position" field in a biography markdown file (located at content/pages/members/) using commas as separators.

Screenshots

people_page

Appearance on people page.

vacancy_page

Appearance on vacancy page (MSc project).

njmhendrix commented 4 years ago

I applied a quick fix for this issue by changing the text line to "position: Associate Professor and Radiologist". I still think it would be nice to be able to use commas, so that's why I haven't closed this issue yet.

martvanrijthoven commented 3 years ago

TODO for mart: check if quotation marks will solve it