Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix Invites & Referrals #1828

Closed
wants to merge 2 commits into from
Closed

Conversation

JisanAR03
Copy link
Contributor

After evaluating our needs for implementing a referral system in our application, decided not to use the django-referral library. The primary reasons are its lack of support for Django 2.0 and several errors that hindered integration.

What We Did:

  • Model Update: We have updated the InviteFriend model to include additional fields such as sender, recipients, referral_code. These enhancements are aimed at tracking invitations more efficiently and enabling the application to monitor the sign-up status of invited users.

  • Security Measures: To safeguard our system against potential spam and abuse, we've implemented several security-related validations. These measures ensure that our invitation system remains secure and is used as intended, promoting a healthy and spam-free environment.

  • Invitation Page Update: Minor improvements were made to the invitation page, enhancing its usability and visual appeal.

  • Reward System Adjustments: We've refined our reward system. Previously, users could earn rewards for merely sending out invitations. Now, rewards are granted only when the invited users complete the signup process successfully. This change aims to encourage genuine invitations and active participation within our community.

Conclusion:
These updates bring our referral system in line with modern standards, ensuring it's both secure and user-friendly. By making these changes, we're not only improving the user experience but also reinforcing the integrity of our platform.

This PR fix issue #1210 #581 .
@DonnieBLT sir, could you please review this PR and share your feedback? Thank you very much.😊

@JisanAR03 JisanAR03 closed this Feb 25, 2024
@JisanAR03 JisanAR03 deleted the fix_issue_1210 branch February 26, 2024 13:03
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