Skip to content

Fixed typo#25840

Open
muratcorlu wants to merge 3 commits intoTryGhost:mainfrom
muratcorlu:patch-5
Open

Fixed typo#25840
muratcorlu wants to merge 3 commits intoTryGhost:mainfrom
muratcorlu:patch-5

Conversation

@muratcorlu
Copy link
Copy Markdown
Contributor

@muratcorlu muratcorlu commented Jan 13, 2026

While searching something related with Signup flow, I noticed that typo (by making same typo in my search). Just a quick fix :)

Got some code for us? Awesome 🎊!

Please take a minute to explain the change you're making:

  • Why are you making it?
  • What does it do?
  • Why is this something Ghost users or developers need?

Please check your PR against these items:

  • I've read and followed the Contributor Guide
  • I've explained my change
  • I've written an automated test to prove my change works

We appreciate your contribution! 🙏


Note

Fixes a typo in portal notifications affecting signup error links.

  • Rename singupPortalLink to signupPortalLink in notification.js and update its usages for signup and signup-paid error messages to ensure the retry link points to the correct signup URL

Written by Cursor Bugbot for commit f239609. This will update automatically on new commits. Configure here.

While searching something related with Signup flow, I noticed that typo (by making same typo in my search). Just a quick fix :)
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 13, 2026

Walkthrough

A typo in the notification.js component is corrected, where the internal variable name singupPortalLink is renamed to signupPortalLink. Two occurrences of this misspelled variable are updated. No changes are made to exported entities, function signatures, or control flow logic. The behavior of the component remains functionally identical after this correction.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fixed typo' directly matches the changeset's primary focus: correcting the variable name typo from 'singupPortalLink' to 'signupPortalLink'.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description clearly explains the change: a typo fix correcting 'singupPortalLink' to 'signupPortalLink' in the notification flow. The description is directly related to the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 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 and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant