Conversation
There was a problem hiding this comment.
Pull request overview
Adds the two missing directors requested in #38 to the localized teamData.json datasets so they appear on the site across PT/ES/EN (with image paths pointing to public/team assets).
Changes:
- Added “Beatriz Fugi” (Marketing Director) to
teamData.jsonin PT/ES/EN. - Added “Gabriel Fagundes” (Projects/Project Director) to
teamData.jsonin PT/ES/EN. - Referenced new director photos via
/team/<file>image paths.
Reviewed changes
Copilot reviewed 3 out of 6 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/i18n/locales/pt/data/teamData.json | Adds the two director entries in Portuguese locale. |
| src/i18n/locales/es/data/teamData.json | Adds the two director entries in Spanish locale. |
| src/i18n/locales/en/data/teamData.json | Adds the two director entries in English locale. |
| "name": "Beatriz Fugi", | ||
| "role": "Diretora de Marketing", | ||
| "image": "/team/beatriz-fugi.jpeg", | ||
| "bio": "Graduanda em Tecnologia em Marketing pela Fundação Getulio Vargas, com bacharelado em Nutrição pela Universidade de São Paulo e especialização em Nutrição, Metabolismo e Fisiologia do Exercício pela FMRP-USP. Atualmente, integro a Diretoria de Marketing da Rede de Avanço em Inteligência Artificial (RAIA), onde produzo materiais para redes sociais.", |
There was a problem hiding this comment.
The new bio appears to contain decomposed/combining diacritics (e.g., "Fundação", "Nutrição"), which is inconsistent with the precomposed accented characters used elsewhere in this JSON. Please normalize this text to standard NFC characters to avoid rendering/search issues and noisy diffs later.
| "bio": "Graduanda em Tecnologia em Marketing pela Fundação Getulio Vargas, com bacharelado em Nutrição pela Universidade de São Paulo e especialização em Nutrição, Metabolismo e Fisiologia do Exercício pela FMRP-USP. Atualmente, integro a Diretoria de Marketing da Rede de Avanço em Inteligência Artificial (RAIA), onde produzo materiais para redes sociais.", | |
| "bio": "Graduanda em Tecnologia em Marketing pela Fundação Getulio Vargas, com bacharelado em Nutrição pela Universidade de São Paulo e especialização em Nutrição, Metabolismo e Fisiologia do Exercício pela FMRP-USP. Atualmente, integro a Diretoria de Marketing da Rede de Avanço em Inteligência Artificial (RAIA), onde produzo materiais para redes sociais.", |
| "name": "Beatriz Fugi", | ||
| "role": "Marketing Director", | ||
| "image": "/team/beatriz-fugi.jpeg", | ||
| "bio": "Undergraduate student in Marketing Technology at Fundação Getulio Vargas, with a Bachelor's degree in Nutrition from the Universidade de São Paulo and a specialization in Nutrition, Metabolism, and Exercise Physiology from FMRP-USP. Currently, I am part of the Marketing Board at the Artificial Intelligence Advancement Network (RAIA), where I produce content for social media.", |
There was a problem hiding this comment.
The new bio contains decomposed/combining diacritics (e.g., "Fundação", "Universidade de São Paulo") rather than the precomposed accented characters used in most of this file. Please normalize to NFC characters to keep text consistent and avoid subtle rendering/search issues.
| "bio": "Undergraduate student in Marketing Technology at Fundação Getulio Vargas, with a Bachelor's degree in Nutrition from the Universidade de São Paulo and a specialization in Nutrition, Metabolism, and Exercise Physiology from FMRP-USP. Currently, I am part of the Marketing Board at the Artificial Intelligence Advancement Network (RAIA), where I produce content for social media.", | |
| "bio": "Undergraduate student in Marketing Technology at Fundação Getulio Vargas, with a Bachelor's degree in Nutrition from the Universidade de São Paulo and a specialization in Nutrition, Metabolism, and Exercise Physiology from FMRP-USP. Currently, I am part of the Marketing Board at the Artificial Intelligence Advancement Network (RAIA), where I produce content for social media.", |
| }, | ||
| { | ||
| "id": "beatriz-fugi", | ||
| "name": "Beatriz Fugi", |
There was a problem hiding this comment.
In the Spanish locale file, the new object starting for Beatriz is indented differently from the other array entries (extra leading spaces before the opening {). This makes the JSON harder to read and breaks the consistent formatting used in the rest of this file; please reformat this block to match the existing indentation (or run the repo’s formatter).
| "name": "Beatriz Fugi", | ||
| "role": "Directora de Marketing", | ||
| "image": "/team/beatriz-fugi.jpeg", | ||
| "bio": "Estudiante de Tecnología en Marketing por la Fundação Getulio Vargas, con licenciatura en Nutrición por la Universidade de São Paulo y especialización en Nutrición, Metabolismo y Fisiología del Ejercicio por la FMRP-USP. Actualmente, integro la Dirección de Marketing de la Rede de Avanço em Inteligência Artificial (RAIA), donde produzco materiales para redes sociales.", |
There was a problem hiding this comment.
This Spanish bio string includes Portuguese terms (e.g., "Universidade de São Paulo" / "Rede de Avanço...") while the rest of the ES locale uses Spanish (e.g., "Universidad..."). Please translate these parts to Spanish or consistently keep official names untranslated across the ES locale to avoid mixed-language content.
| "bio": "Estudiante de Tecnología en Marketing por la Fundação Getulio Vargas, con licenciatura en Nutrición por la Universidade de São Paulo y especialización en Nutrición, Metabolismo y Fisiología del Ejercicio por la FMRP-USP. Actualmente, integro la Dirección de Marketing de la Rede de Avanço em Inteligência Artificial (RAIA), donde produzco materiales para redes sociales.", | |
| "bio": "Estudiante de Tecnología en Marketing por la Fundación Getulio Vargas, con licenciatura en Nutrición por la Universidad de São Paulo y especialización en Nutrición, Metabolismo y Fisiología del Ejercicio por la FMRP-USP. Actualmente, formo parte de la Dirección de Marketing de la Red de Avance en Inteligencia Artificial (RAIA), donde produzco materiales para redes sociales.", |
📌 Summary
This PR adds data and photos for two missing directors.
✨ Changes Introduced
🔗 Related Issues