IMGIITRoorkee / IMGIITRoorkee/placement-flutter

Show a real reason for ineligible profiles, not a no-signal icon

Open Beginner friendly
#58 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug P2
Dominant language
Dart
Stars
3
Forks
10
Avg merge
8d 21h
Merged PRs (30d)
4

Description

Problem

The backend defines seven profileStatus values. The icon switch handles five, and the two it misses fall through to the default: branch, which renders a "no mobile signal" icon.

So a student who is short on CGPA, or is ineligible for the profile, is shown an icon telling them their network is down.

ModelResources.analyseProfileStatus already maps both statuses to correct English and is simply not used at these two sites.

Where

  • lib/shared/ProfileStatusIcon.dart:143-144 (default: renders Icons.signal_cellular_connected_no_internet_4_bar)
  • lib/views/CompanyDetailView.dart:518-519 (same)
  • lib/resources/modelResources.dart:19-28 (the correct mapping, unused here)
  • Backend statuses: omniport-app-placement-and-internship/constants/profile_statuses.py:4-27

Expected

candidate_ineligible and cgpa_not_eligible show a neutral info icon with the real reason.

Fix sketch

  • Route the default: branch through analyseProfileStatus with an info icon.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in lib/shared/ProfileStatusIcon.dart:143-144 and lib/views/CompanyDetailView.dart:518-519, then read the existing mapping in lib/resources/modelResources.dart:19-28 and the backend statuses in constants/profile_statuses.py:4-27. Update the two default cases so candidate_ineligible and cgpa_not_eligible use a neutral info icon and the mapped reason. Done means both sites show the real English reason instead of the no-signal icon.

Written by the indexing model from the issue text.

Assessment

Tech stack
dart, flutter
Domain
mobile
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
86/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.