feat: personal info by muralhaplays · Pull Request #1799 · NIAEFEUP/uni

muralhaplays · 2026-02-23T11:21:51Z

Closes #1774

Review checklist

Terms and conditions reflect the changes

View Changes

Description has screenshots of the UI changes.
Tested both in light and dark mode.
New text is both in portuguese (PT) and english (EN).
Works in different text zoom levels.
Works in different screen sizes.

Performance

No helper functions to return widgets are added. New widgets are created instead.
Used ListView.builder for Long Lists.
Controllers (TextEditingController, ...) are beeing disposed of in dispose() method.

Before

After

…rofile class.

…rofile info page.

…n expected. I hate SIGARRA!

…page. Change the visual aspect of the new page.

pedroafmonteiro

Waiting for conflict resolution!

codecov · 2026-03-05T08:59:22Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 14%. Comparing base (d2e3b63) to head (abbfa47).
⚠️ Report is 1 commits behind head on develop.

❌ Your project check has failed because the head coverage (14%) is below the target coverage (70%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #1799    +/-   ##
========================================
+ Coverage        7%     14%    +7%     
========================================
  Files           38      35     -3     
  Lines         1886     931   -955     
========================================
- Hits           125     124     -1     
+ Misses        1761     807   -954

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot

Pull request overview

Implements the “Personal Information” feature (closes #1774) by fetching personal profile details from Sigarra and exposing them via a new profile-info page reachable from the profile settings.

Changes:

Added profile-info fetching/parsing and new UI pages/widgets to display personal information sections.
Added navigation entry + icon to reach the new page, and registered the route in the app router.
Updated localization strings and regenerated ObjectBox / intl artifacts.

Reviewed changes

Copilot reviewed 25 out of 32 changed files in this pull request and generated 17 comments.

Show a summary per file

File	Description
packages/uni_ui/lib/icons.dart	Adds a new user/profile icon constant for UI navigation.
packages/uni_ui/lib/cards/data_list_tile.dart	Introduces a reusable list-tile for key/value personal info rows.
packages/uni_app/lib/view/widgets/pages_layouts/general/widgets/profile_button.dart	Tweaks profile button avatar radius.
packages/uni_app/lib/view/profile_info/widgets/profile_overview.dart	Adds header/overview widget for the profile-info page.
packages/uni_app/lib/view/profile_info/widgets/profile_info_shimmer.dart	Adds shimmer loading skeleton for the profile-info page.
packages/uni_app/lib/view/profile_info/widgets/profile_data.dart	Adds UI for rendering personal info sections into cards.
packages/uni_app/lib/view/profile_info/widgets/no_profile_data.dart	Adds empty-state widget for missing profile-info data.
packages/uni_app/lib/view/profile_info/profile_info.dart	Adds the profile-info page (Secondary page layout + refresh).
packages/uni_app/lib/view/profile/widgets/settings.dart	Adds a settings tile to navigate to Personal Information.
packages/uni_app/lib/view/profile/widgets/profile_info.dart	Renames `ProfileInfo` widget to `ProfileInfoWidget`.
packages/uni_app/lib/view/profile/profile.dart	Updates profile page to use the renamed widget.
packages/uni_app/lib/view/home/home.dart	Moves locale-change-driven profile refresh to Home page lifecycle.
packages/uni_app/lib/view/academic_path/courses_page.dart	Removes locale-change-driven profile refresh from Courses page.
packages/uni_app/lib/utils/navigation_items.dart	Adds a new navigation route entry for profile-info.
packages/uni_app/lib/objectbox.g.dart	Regenerated ObjectBox bindings/model for entity changes.
packages/uni_app/lib/objectbox-model.json	Updates ObjectBox model JSON (entity/property metadata).
packages/uni_app/lib/model/entities/profile_info.dart	Adds the `ProfileInfo` entity/model to hold personal info.
packages/uni_app/lib/model/entities/profile.dart	Adds `profileInfo` field to `Profile` model.
packages/uni_app/lib/main.dart	Registers the new profile-info route in the app router.
packages/uni_app/lib/l10n/intl_pt_PT.arb	Adds PT translations for new Personal Information UI strings.
packages/uni_app/lib/l10n/intl_en.arb	Adds EN translations for new Personal Information UI strings.
packages/uni_app/lib/generated/l10n.dart	Regenerated localization accessors (intl).
packages/uni_app/lib/generated/intl/messages_pt_PT.dart	Regenerated PT message lookup table.
packages/uni_app/lib/generated/intl/messages_en.dart	Regenerated EN message lookup table.
packages/uni_app/lib/controller/parsers/parser_profile_info.dart	Adds HTML parser for profile personal-information tables.
packages/uni_app/lib/controller/fetchers/profile_info_fetcher.dart	Adds network fetcher for Sigarra personal information.
packages/uni_app/lib/controller/fetchers/profile_fetcher.dart	Extends profile fetch to also retrieve profile-info data.
lib/l10n/intl_en.arb	Adds (empty) root-level localization ARB file.
lib/generated/l10n.dart	Adds root-level generated localization delegate/accessors.
lib/generated/intl/messages_en.dart	Adds root-level generated EN messages file.
lib/generated/intl/messages_all.dart	Adds root-level generated messages loader.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-08T18:15:37Z

+  "contacts": "Contactos",
+  "@contacts": {},


intl_pt_PT.arb defines the key contacts twice (first as "Contactos" and later as "Contactos Gerais"). Duplicate JSON keys are invalid/ambiguous and will result in one value overriding the other during generation. Split these into distinct keys (e.g. contacts vs general_contacts) or remove the duplicate so each message key is unique.

Suggested change

"contacts": "Contactos",

"@contacts": {},

"contacts_label": "Contactos",

"@contacts_label": {},

Copilot · 2026-03-08T18:15:37Z

+        fontSize: 18,
+        color: Theme.of(context).colorScheme.primary,
+      ),
+      sublabel: S.of(context).no_courses_description,


NoProfileDataWidget uses S.of(context).no_courses_description as the sublabel, which is unrelated to the “no personal info” empty state. This will show the wrong message to users; consider adding/using a dedicated localization key describing the absence of personal information.

Suggested change

sublabel: S.of(context).no_courses_description,

sublabel: '',

Copilot · 2026-03-08T18:15:37Z

+    return ProfileInfo(
+      profileInfo: list[0],
+      nationalities: list[1],
+      identification: list[2],
+      contacts: list[3],
+      addresses: list[4],


ProfileInfo.fromList indexes list[0]..list[4] without validating that those indices exist. If parsing returns fewer tables (or the order changes), this will throw a RangeError. Add length checks and sensible defaults (or map by section identifiers instead of raw indices).

Suggested change

return ProfileInfo(

profileInfo: list[0],

nationalities: list[1],

identification: list[2],

contacts: list[3],

addresses: list[4],

final List<List<String>> profileInfoSection =

list.length > 0 ? list[0] : const <List<String>>[];

final List<List<String>> nationalitiesSection =

list.length > 1 ? list[1] : const <List<String>>[];

final List<List<String>> identificationSection =

list.length > 2 ? list[2] : const <List<String>>[];

final List<List<String>> contactsSection =

list.length > 3 ? list[3] : const <List<String>>[];

final List<List<String>> addressesSection =

list.length > 4 ? list[4] : const <List<String>>[];

return ProfileInfo(

profileInfo: profileInfoSection,

nationalities: nationalitiesSection,

identification: identificationSection,

contacts: contactsSection,

addresses: addressesSection,

Copilot · 2026-03-08T18:15:37Z

+  /// Returns a list with two tuples: the first tuple contains the user's name
+  /// and the other one contains the user's email.


The keymapValues doc comment says it returns “two tuples” (name + email), but the method returns multiple entries (profileInfo, nationalities, identification, contacts, addresses). Update the comment to reflect the actual return value.

Suggested change

/// Returns a list with two tuples: the first tuple contains the user's name

/// and the other one contains the user's email.

/// Returns a list of key/value tuples for this profile, where each tuple

/// contains the field name and its corresponding list of values.

pedroafmonteiro

Thanks for the continued work on this PR! Consider some last changes please:

There are some renderbox error logs when clicking on the personal information tile.
With some updates to develop, there's a new profile info shimmer that should be used.
The "Personal Information" button should be positioned above the "Settings" heading, so it actually belongs to the Profile section of the page.
On first load, the page freezes a bit. The loading state should be here to symbolize loading.

Thanks again!

… into feat/personal_info

muralhaplays added 7 commits February 11, 2026 20:19

Add personal information to settings page. Add new atributes to the p…

94d7863

…rofile class.

Add new attributes to profile. Fetch the attributes that were added.

97ba368

Format the files

6540a69

Change profile to store the type of the address as well. Create the p…

334be34

…rofile info page.

Rebuild everything because the sigarra page can have more entries tha…

9e63020

…n expected. I hate SIGARRA!

Format a few things and remove unused imports

79b9c31

Restore the duration of profile cache

df9e586

pedroafmonteiro linked an issue Feb 25, 2026 that may be closed by this pull request

Show Personal Information on the app #1774

Open

muralhaplays added 2 commits February 25, 2026 16:09

Change the didChangeDependencies() from the courses page to the home …

6c9f56f

…page. Change the visual aspect of the new page.

Change the profile button

d21890a

muralhaplays requested a review from a team February 26, 2026 15:18

pedroafmonteiro requested changes Mar 1, 2026

View reviewed changes

pedroafmonteiro assigned muralhaplays Mar 1, 2026

muralhaplays added 4 commits March 2, 2026 13:52

Merge branch 'develop' into feat/personal_info

2349b62

formatted

c269147

Add shimmer and no content widgets to the personal info page.

2335518

Format files

7068f1f

pedroafmonteiro requested a review from Copilot March 8, 2026 18:06

Copilot started reviewing on behalf of pedroafmonteiro March 8, 2026 18:07 View session

Copilot AI reviewed Mar 8, 2026

View reviewed changes

muralhaplays added 4 commits March 8, 2026 18:34

Remove a folder that was mistakenly created

bae44e3

Merge branch 'develop' into feat/personal_info

78e69e2

Change the way we save the profile info

b5db565

Put the ProfileInfo into the database

2fea347

pedroafmonteiro changed the title ~~Feat/personal info~~ feat: personal info Mar 15, 2026

muralhaplays and others added 3 commits March 23, 2026 17:05

Fix copilot suggestions. Save profile info into the database

1ea25ae

Change profile info cache duration to 30 days

3232ce3

Merge branch 'develop' into feat/personal_info

6c95cb8

pedroafmonteiro requested changes Mar 31, 2026

View reviewed changes

muralhaplays requested a review from pedroafmonteiro April 20, 2026 16:00

muralhaplays and others added 4 commits April 20, 2026 17:00

Merge branch 'develop' into feat/personal_info

ac00ee0

Change the personal info button to above the settings heading

5d74d9c

Merge remote-tracking branch 'refs/remotes/origin/feat/personal_info'…

4258db6

… into feat/personal_info

Fix the request changes

abbfa47

muralhaplays requested a review from a team April 22, 2026 14:56

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: personal info#1799

feat: personal info#1799
muralhaplays wants to merge 24 commits intodevelopfrom
feat/personal_info

muralhaplays commented Feb 23, 2026 •

edited by pedroafmonteiro

Loading

Uh oh!

pedroafmonteiro left a comment

Uh oh!

codecov Bot commented Mar 5, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Mar 8, 2026

Uh oh!

Copilot AI Mar 8, 2026

Uh oh!

Copilot AI Mar 8, 2026

Uh oh!

Copilot AI Mar 8, 2026

Uh oh!

pedroafmonteiro left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	sublabel: S.of(context).no_courses_description,
	sublabel: '',

-    return ProfileInfo(
-      profileInfo: list[0],
-      nationalities: list[1],
-      identification: list[2],
-      contacts: list[3],
-      addresses: list[4],
+    final List<List<String>> profileInfoSection =
+        list.length > 0 ? list[0] : const <List<String>>[];
+    final List<List<String>> nationalitiesSection =
+        list.length > 1 ? list[1] : const <List<String>>[];
+    final List<List<String>> identificationSection =
+        list.length > 2 ? list[2] : const <List<String>>[];
+    final List<List<String>> contactsSection =
+        list.length > 3 ? list[3] : const <List<String>>[];
+    final List<List<String>> addressesSection =
+        list.length > 4 ? list[4] : const <List<String>>[];
+    return ProfileInfo(
+      profileInfo: profileInfoSection,
+      nationalities: nationalitiesSection,
+      identification: identificationSection,
+      contacts: contactsSection,
+      addresses: addressesSection,

		/// Returns a list with two tuples: the first tuple contains the user's name
		/// and the other one contains the user's email.

Conversation

muralhaplays commented Feb 23, 2026 • edited by pedroafmonteiro Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review checklist

View Changes

Performance

Before

After

Uh oh!

pedroafmonteiro left a comment

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

Uh oh!

pedroafmonteiro left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

muralhaplays commented Feb 23, 2026 •

edited by pedroafmonteiro

Loading

codecov Bot commented Mar 5, 2026 •

edited

Loading