Skip to content

fix: add null name guard in recommendations.js - #7539

Merged
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7518-recommendations-null-name-guard
Aug 13, 2026
Merged

fix: add null name guard in recommendations.js#7539
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7518-recommendations-null-name-guard

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

📄 Description

Added a filter to getRecommendations to exclude history items with null/undefined names or ids, preventing broken recommendation cards from rendering.

🔗 Related Issues

Closes #7518

🧩 Type of Change

  • [[x]] Bug fix
  • [] New feature
  • [] Documentation update
  • [] Style / UI improvement
  • [] Refactor
  • [] Test
  • [] Chore / dependency update

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Note: This task is being handled by tmdeveloper007 as part of GSSOC (GirlScript Summer of Code) — please assign this PR to the tmdeveloper007 account when picking it up.

@tmdeveloper007 tmdeveloper007 changed the title fix : add null name guard in recommendations.js fix: add null name guard in recommendations.js Aug 12, 2026
@janavipandole
janavipandole merged commit 837ed9c into janavipandole:main Aug 13, 2026
21 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix : add null guard for product name in recommendations.js

2 participants