Skip to content

Answer: 57 Used content projection - #1553

Open
jpaberzs wants to merge 1 commit into
tomalaforge:mainfrom
jpaberzs:angular-content-projection-default
Open

Answer: 57 Used content projection#1553
jpaberzs wants to merge 1 commit into
tomalaforge:mainfrom
jpaberzs:angular-content-projection-default

Conversation

@jpaberzs

@jpaberzs jpaberzs commented Sep 3, 2026

Copy link
Copy Markdown

✅ Challenge Submission Checklist

Start your PR title with: Answer:${challenge_number}

⚠️ Important Notice

If you would like personal feedback or a detailed review, please support the project on GitHub:

You can also submit a PR without sponsorship to:

  • Be listed among the answered challenges, or
  • Receive a review from a community member. 🔥

Summary by CodeRabbit

  • New Features
    • Cards now support content projection, allowing titles and messages to be provided as child content.
    • Cards display a default “Aucun message” message when no message content is supplied.

@vercel

vercel Bot commented Sep 3, 2026

Copy link
Copy Markdown

@jpaberzs is attempting to deploy a commit to the tomalaforge's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: de535bcd-d3d8-42eb-b0f9-d34d96a47d7b

📥 Commits

Reviewing files that changed from the base of the PR and between 0deb691 and d8ea9d5.

📒 Files selected for processing (2)
  • apps/angular/57-content-projection-default/src/app/app.component.ts
  • apps/angular/57-content-projection-default/src/app/card.component.ts

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The card component now uses Angular content projection for titles and messages. The app component projects content into two cards. Cards without a projected message display “Aucun message”.

Changes

Content projection

Layer / File(s) Summary
Projection contract and card usage
apps/angular/57-content-projection-default/src/app/card.component.ts, apps/angular/57-content-projection-default/src/app/app.component.ts
CardComponent replaces signal inputs with [title] and [message] content slots. The message slot provides the “Aucun message” fallback. AppComponent projects title and message elements into the first card and only a title element into the second card.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to d8ea9

Cards now receive projected title and message content, with a fallback message when none is provided. The updated card usage matches this behavior and no current merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description includes the required challenge submission checklist and sponsorship notice. It matches the repository template.
Title check ✅ Passed The title starts with "Answer: 57" and clearly identifies the use of content projection for challenge 57.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2 files.

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Warning

⚠️ This pull request has been flagged as potential spam (promotional) by CodeRabbit slop detection and should be reviewed carefully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

57 ng-content default answer answer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant