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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Project complated : fix invite and referrals #1830

Merged
merged 8 commits into from
Feb 26, 2024

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
Copy link
Contributor Author

@DonnieBLT finally sir , I'm able to make this PR without any error ,could you please review this PR and share your feedback?

Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

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

looks good, thank you!

@DonnieBLT DonnieBLT merged commit 7482fd3 into OWASP-BLT:main Feb 26, 2024
7 checks passed
@JisanAR03
Copy link
Contributor Author

looks good, thank you!

thanks sir

@JisanAR03 JisanAR03 deleted the issue_1210_again_fix branch February 26, 2024 11:49
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.

None yet

2 participants