Skip to content

feat: adiciona grid de membros por categoria - #51

Open
shirubaarison wants to merge 2 commits into
developfrom
feature/grid-membros-por-categoria
Open

shirubaarison wants to merge 2 commits into
developfrom
feature/grid-membros-por-categoria

Conversation

@shirubaarison

Copy link
Copy Markdown
Collaborator

gerar automaticamente um grid de membros pelo slug de uma categoria, evitando manter manualmente a lista de slugs dos integrantes.

exemplo: [membros_categoria categoria="llabtisocial"]

  • categoria: slug obrigatório do setor ou categoria.
  • tipo: filtro opcional. Aceita:
    • atuais — padrão; exige também o termo membro-atual.
    • egressos — filtra pelo termo egresso.
    • lideres — filtra pelo termo lider ou seus descendentes.
    • todos — não filtra pela situação, mas mantém a categoria.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The default atuais filter does not include descendant taxonomy terms, potentially omitting current members.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds the [membros_categoria] shortcode to generate member grids by category with optional status filters.

Changes:

  • Extracts reusable member-grid rendering.
  • Supports current members, alumni, leaders, and all members.
  • Preserves the existing manual [membros] shortcode.
File Summary
themes/​learninglab-site/​inc/​shortcodes.php Adds reusable rendering and taxonomy-based member filtering.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread themes/learninglab-site/inc/shortcodes.php Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

Não há questões pendentes identificadas.

Review effort: Lite
Findings: None

Resolved since last review (1)

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.

2 participants