Skip to content

Missing button for special cases - #625

Merged
mariana2103 merged 13 commits into
developfrom
missingButtonForSpecialCases
May 14, 2026
Merged

Missing button for special cases#625
mariana2103 merged 13 commits into
developfrom
missingButtonForSpecialCases

Conversation

@PedroLunet

Copy link
Copy Markdown
Collaborator

Fix: Handle extraordinary students and add course labels to "Tratar" buttons

Fixes #555

What changed

  • Fixed null reference crashes in AdminMarketplaceExchangesCard when class info is missing
  • "Tratar" button now appears for extraordinary students (not enrolled in L.EIC/M.EIC) using fallback Sigarra URL
  • When multiple courses are involved, buttons show course labels: "Tratar (L.EIC)", "Tratar (M.EIC)"
  • Deduplicated course IDs to prevent duplicate buttons

How it works

  • TreatExchangeButton detects courses without student metadata and generates fallback links to it_adm.adm_t_livre_escolhe_aluno
  • Added courseInfo prop throughout the admin card components to pass course acronyms for labeling
  • Only shows course labels when 2+ courses are present, keeps it simple otherwise

@PedroLunet
PedroLunet requested a review from a team February 13, 2026 17:34
@netlify

netlify Bot commented Feb 13, 2026

Copy link
Copy Markdown

Deploy Preview for tts-fe-preview ready!

Name Link
🔨 Latest commit f7814d1
🔍 Latest deploy log https://app.netlify.com/projects/tts-fe-preview/deploys/69e8f1c72e92e30008d207d7
😎 Deploy Preview https://deploy-preview-625--tts-fe-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown

@mariana2103 mariana2103 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good work! Ready to merge once that small inconsistency is fixed.

Comment thread src/components/admin/requests/cards/StudentEnrollmentCard.tsx
@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

🌐 Netlify Preview disponível:
https://69d61009e01fc50008be6c4d--tts-fe-preview.netlify.app

@github-actions

Copy link
Copy Markdown

🌐 Netlify Preview disponível:
https://69e8f1c72e92e30008d207d7--tts-fe-preview.netlify.app

@mariana2103
mariana2103 merged commit 3155c82 into develop May 14, 2026
8 checks passed
@mariana2103
mariana2103 deleted the missingButtonForSpecialCases branch May 14, 2026 14:06
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.

fix: missing “Tratar” button for special cases

2 participants