Skip to content

feat: add error explained tutorial template - #286

Merged
anurag629 merged 2 commits into
codercops:devfrom
anurag2787:add/template-error
Oct 27, 2026
Merged

anurag629 merged 2 commits into
codercops:devfrom
anurag2787:add/template-error

Conversation

@anurag2787

@anurag2787 anurag2787 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Description

Added the tutorial-error-explained template to show an error message

Closes #283

How I Tested This

Ran npm run check and npm run test

AI Assistance

ChatGPT was used to help with the implementation and tests

Type of Change

  • Bug fix
  • New template
  • New feature
  • Documentation
  • Tests
  • Refactoring
  • Performance improvement
  • Other (describe below)

Screenshots

image image

Checklist

  • This PR targets dev
  • I was assigned the linked issue
  • npm run check passes
  • npm run test passes
  • For a bug fix: the new test fails without my change
  • Tested locally with npm run dev
  • Screenshots included for visual changes
  • New templates follow the Template Guidelines

Signed-off-by: Anurag Yadav <anuragyadav2787@gmail.com>
@anurag2787

Copy link
Copy Markdown
Contributor Author

Hi @anurag629 could you please review and let me know if any changes required
Thanks!

@anurag629 anurag629 left a comment

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.

Thanks @anurag2787, this is close. Check, all 1429 tests and the build pass on top of dev, and the card looks right with the defaults, on a white background with a pale accent, with a long error message and with blank fields.

One thing: when the meaning or the fix runs to two lines, its label lines up with the last line instead of the first. With a meaning like "You called .map() on a value that is still undefined, usually because the data has not loaded yet", MEANING sits next to "has not loaded yet". The two rows use alignItems: 'baseline' (lines 99 and 142), and in the rendered card that puts the label next to the last line. alignItems: 'flex-start' with a few pixels of top margin on the label would keep it next to the first line. Could you change that?

Signed-off-by: Anurag Yadav <anuragyadav2787@gmail.com>
@anurag2787

Copy link
Copy Markdown
Contributor Author

Hi @anurag629 i have updated PR could you please re review
Thanks!

@anurag629 anurag629 left a comment

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.

Thanks @anurag2787, that fixes it. Check, all 1443 tests and the build pass on top of dev, and with a two-line meaning and fix the labels now sit next to the first line.

@anurag629 anurag629 added the hacktoberfest-accepted PR counts for Hacktoberfest label Oct 27, 2026
@anurag629
anurag629 merged commit 1363a09 into codercops:dev Oct 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted PR counts for Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the Error Explained tutorial template (tutorial-error-explained)

2 participants