UTDNebula/utd-trends

[Feature Request] More specific RMP summary errors

Ouverte

#610 ouverte le 7 avr. 2026

 (6 commentaires) (1 réaction) (1 personne assignée)TypeScript (35 forks)auto 404
Good First IssueType: Feature Request

Métriques du dépôt

Stars
 (28 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Overview

Currently the RMP summary route can return errors for things like API keys not being defined, but also for there not being any reviews on a RMP page. All these errors display as "Problem loading AI RMP Summary". See: https://trends.utdnebula.com/dashboard?searchTerms=Tahrima+Rahman&availability=26F which only has 2 reviews so in src\app\api\rmpSummary\route.ts the below is returned:

if (rmp.ratings.edges.length < 5) {
  return NextResponse.json(
    { message: 'error', data: 'Not enough ratings for a summary' },
    { status: 500 },
  );
}

but in src\components\common\RmpSummary\RmpSummary.tsx only the below is rendered.

if (status === 'error') {
  return <p>Problem loading AI review summary.</p>;
}

Proposal

Update the error display of the RmpSummary component to read the specific error returned from the API and display a more descriptive error message. We can leave our internal errors like missing API key and incorrect query parameters as "Problem loading AI RMP Summary". But the two RMP related errors should be more descriptive.

Guide contributeur