Skip to content

Conversation

galalqassas
Copy link
Collaborator

Quality Checklist

  • I took the time to compare my implementation with the design.
  • I ensured that the this pull request satisfies all the requirements in the associated task.
  • I performed a self-review.

Links

Important

Incase you have any valuable knowledge that you think that it is worth writing down, feel free to add it to our handbook. Wounder why we are doing this? check this guide.

@galalqassas galalqassas requested a review from moalidv September 15, 2025 19:20
@galalqassas galalqassas self-assigned this Sep 15, 2025
@galalqassas galalqassas marked this pull request as ready for review September 15, 2025 19:20
@neuodev
Copy link
Member

neuodev commented Sep 15, 2025

🚀 WEB
🚀 UI
🚀 LANDING
🚀 DASHBOARD

@galalqassas galalqassas force-pushed the galal/redesign-lesson-card-component branch from 51b1e33 to fc60c8c Compare September 15, 2025 19:29
@neuodev
Copy link
Member

neuodev commented Sep 15, 2025

🚀 WEB
🚀 UI
🚀 LANDING
🚀 DASHBOARD

@galalqassas galalqassas force-pushed the galal/redesign-lesson-card-component branch from fc60c8c to c1aa29f Compare September 15, 2025 19:41
@neuodev
Copy link
Member

neuodev commented Sep 15, 2025

Warnings
⚠️ No clickup task found in this pull request

Generated by 🚫 dangerJS against 3802386

Copy link
Collaborator

@mmoehabb mmoehabb left a comment

Choose a reason for hiding this comment

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

So good so far. However you need to pay more attention to details. You must review your work first to make sure it's okay.

Spot the differences between the prototype and your implementation and work on it:

The implementation:
Screenshot_20250916_164652

The prototype:
Screenshot_20250916_164919

@galalqassas galalqassas force-pushed the galal/redesign-lesson-card-component branch from c1aa29f to b7205c8 Compare September 24, 2025 17:22
@galalqassas galalqassas force-pushed the galal/redesign-lesson-card-component branch from b7205c8 to 3802386 Compare September 24, 2025 18:19
Comment on lines +177 to +178
topics: range(7).map(() => faker.lorem.words(1)),
level: "C1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

There is no English level for tutors. This should only appear in students cards for tutors.
Furthermore, lesson-card for students is different from lesson-card for tutors.

Copy link
Collaborator

Choose a reason for hiding this comment

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

You better write this in LessonCardV2 for instance.

<Typography
key="remaining"
tag="span"
className="border border-natural-500 rounded-[200px] px-1.5 py-1"
Copy link
Collaborator

Choose a reason for hiding this comment

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

specify the text color, size, and width to match the prototype.

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.

3 participants