Skip to content

feat: add course enrollment fetching data & fix UI#5

Open
bra-i-am wants to merge 7 commits intobc/implement-new-uifrom
bc/use-real-data
Open

feat: add course enrollment fetching data & fix UI#5
bra-i-am wants to merge 7 commits intobc/implement-new-uifrom
bc/use-real-data

Conversation

@bra-i-am
Copy link

@bra-i-am bra-i-am commented Jan 31, 2026

Description

This PR adds a new endpoint to fetch the real data that should be displayed in the course cards at "/catalog/:org/:key/*"

Screenshot

image image

…t strings

- Integrated `useIntl` from `@edx/frontend-platform/i18n` across multiple components including CourseCard, CourseDetails, Dashboard, DataSharingAuthorizationModal, FilterPanel, LearningPathCard, and LearningPathDetails.
- Created a new messages file to define all text strings used in the learning path components for better localization support.
- Replaced hardcoded strings with formatted messages to enhance maintainability and support for multiple languages.
@bra-i-am bra-i-am changed the title feat: add course enrollment fetching and update CourseCard to display… feat: add course enrollment fetching data & fix UI Feb 2, 2026
@bra-i-am bra-i-am marked this pull request as ready for review February 2, 2026 15:53
@bra-i-am bra-i-am requested a review from dcoa February 2, 2026 15:53
@bra-i-am bra-i-am marked this pull request as draft February 2, 2026 15:58
@bra-i-am bra-i-am marked this pull request as ready for review February 3, 2026 13:35
Copy link

@dcoa dcoa left a comment

Choose a reason for hiding this comment

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

This LGTM, just take in consideration the lint problem to be solver either here or as part of the other PR.

@bra-i-am bra-i-am force-pushed the bc/implement-new-ui branch from d838e03 to d089dde Compare February 5, 2026 21:51
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