Feature/novos diretores/Reordered director list to position Gabriel Fagundes next to other Project Directors#44
Merged
Augusto-Ildefonso merged 4 commits intomainfrom Mar 19, 2026
Conversation
There was a problem hiding this comment.
Pull request overview
Reorders the localized team member lists so that Gabriel Fagundes appears alongside the other Project/Projects Directors in the team roster data used by the site’s i18n-driven team pages.
Changes:
- Reordered entries in
teamData.jsonfor pt, es, and en locales to movegabriel-fagundesnext to other project directors. - No content changes intended beyond the ordering (same member data, different position).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/i18n/locales/pt/data/teamData.json |
Moves gabriel-fagundes entry earlier to align with other “Diretor de Projetos” entries. |
src/i18n/locales/es/data/teamData.json |
Moves gabriel-fagundes entry earlier to align with other “Director de Proyectos” entries. |
src/i18n/locales/en/data/teamData.json |
Moves gabriel-fagundes entry earlier to align with other project directors (but role string is inconsistent—see comment). |
| { | ||
| "id": "gabriel-fagundes", | ||
| "name": "Gabriel Fagundes", | ||
| "role": "Project Director", |
There was a problem hiding this comment.
In the EN locale, this member’s role is "Project Director" while the other project directors in the same list use "Projects Director". This inconsistency will show up in the UI and can also break any role-based grouping/filtering in the future. Consider standardizing the role string (e.g., change this to "Projects Director" or align all entries to a single form).
Suggested change
| "role": "Project Director", | |
| "role": "Projects Director", |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Summary
Reordered director list to position Gabriel Fagundes next to other Project Directors.
✨ Changes Introduced