Skip to content

fix: loyalty points awarded 100x lower than configured POINTS_PER_RUPEE (#5983) - #6522

Merged
janavipandole merged 1 commit into
janavipandole:mainfrom
adrika4:feat/5707-gift-msg-length-validation
Aug 10, 2026
Merged

fix: loyalty points awarded 100x lower than configured POINTS_PER_RUPEE (#5983)#6522
janavipandole merged 1 commit into
janavipandole:mainfrom
adrika4:feat/5707-gift-msg-length-validation

Conversation

@adrika4

@adrika4 adrika4 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

📋 Summary

Fixes loyalty points calculation so customers are awarded the correct
number of points per rupee spent, instead of 100x fewer than configured.


🔗 Related Issue

Closes #5983


🔄 Type of Change

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

🛠️ What Was Changed

  • Corrected the loyalty points calculation in checkout.js so it matches the configured POINTS_PER_RUPEE value
  • Points awarded per order now reflect the intended rate instead of being scaled down by 100x

why this needed

  • Customers were being under-rewarded, receiving 100x fewer loyalty points than the configured rate intended
  • This directly affects redemption value and customer trust in the loyalty program

🧪 How Has This Been Tested?

  • Ran frontend locally with npm run dev
  • Ran npm run lint — no errors
  • Ran npm run build — no errors
  • Uploaded a test document and verified output
  • Tested on mobile view

✅ Self-Review Checklist

  • My branch name follows the convention (feat/, fix/, docs/)
  • My commit messages follow the convention (feat:, fix:, docs:)
  • I have linked the related issue (Closes #IssueNumber)
  • I have tested my changes locally
  • I have added screenshots for UI changes
  • My changes do not break existing functionality
  • I have not pushed any .env file or API keys
  • My PR contains only changes related to the linked issue

📋 Additional Notes

None.

Copilot AI lite review requested due to automatic review settings August 9, 2026 17:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@janavipandole
janavipandole merged commit 9ae0a33 into janavipandole:main Aug 10, 2026
29 of 43 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.

[Bug]: Loyalty points are awarded 100x lower than the configured POINTS_PER_RUPEE rate

3 participants