Skip to content

Feat/teammate card#80

Open
den987655 wants to merge 5 commits into
mainfrom
feat/teammate-card
Open

Feat/teammate card#80
den987655 wants to merge 5 commits into
mainfrom
feat/teammate-card

Conversation

@den987655
Copy link
Copy Markdown
Collaborator

@den987655 den987655 commented May 30, 2026

📝 Summary of Changes

Added reusable TeammateCard component for the About page.
Implemented flip-card interaction with front/back sides on hover.
Added configurable inputs for teammate information:

  • name
  • role
  • description
  • promoCode
  • imgSrc
  • linkHref

Implemented card layout and styling using BEM structure.
Added support for external profile links.

🖼️ UI Changes (Mandatory for frontend/UI tasks)

Снимок экрана 2026-05-30 163304 Снимок экрана 2026-05-30 163314

🛠️ Pre-Review Checklist

  • I have set Assignees, Labels, and linked the GitHub Projects issue in the right sidebar
  • My branch is up to date with main
  • The CI pipeline is passing successfully locally (npm run tsc, npm run lint, etc.)
  • The application boots up locally and has been manually verified

👥 Reviewer Checklist

  • Mentor Assigned: @mummick
  • Second Reviewer Assigned
  • I understand comment prefixes: [praise], [nit], [question] (unprefixed comments = hard blockers)

@den987655 den987655 self-assigned this May 30, 2026
@den987655 den987655 added enhancement New feature or request ui kit labels May 30, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in shop-front May 30, 2026
@den987655 den987655 linked an issue May 30, 2026 that may be closed by this pull request
@elrouss elrouss added about and removed ui kit labels May 30, 2026
Copy link
Copy Markdown
Member

@elrouss elrouss left a comment

Choose a reason for hiding this comment

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

Overall, the PR looks solid!

I've left a few minor comments to fix. As a general piece of feedback, it looks like the default test file teamate-card.component.spec.ts is missing. Please make sure to include it

@@ -0,0 +1,26 @@
<article class="card">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

praise: Great choice of a semantic tag!

Comment thread src/app/features/about/components/teammate-card/teammate-card.component.html Outdated
Comment thread src/app/features/about/components/teammate-card/teammate-card.component.html Outdated
Comment thread src/app/features/about/components/teammate-card/teammate-card.component.html Outdated
Comment thread src/app/features/about/components/teammate-card/teammate-card.component.html Outdated
Comment thread src/app/features/about/components/teammate-card/teammate-card.component.scss Outdated
@elrouss elrouss removed this from shop-front May 30, 2026
@den987655 den987655 force-pushed the feat/teammate-card branch from a366531 to d58ed91 Compare May 31, 2026 11:04
@den987655 den987655 requested a review from elrouss May 31, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

about enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Teammate component for About page

2 participants