-
Notifications
You must be signed in to change notification settings - Fork 73
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
Allow referral in query string of URL #29
Comments
Thanks so much for your suggestion @kevinmickey! Any thoughts about this @paltman and @jtauber? |
Is there any update for this enhancement? I am using this lib from last one year and now I need this feature. @ossanna16, @kevinmickey |
@surinderdeveloper This issue was opened quite a while before I began maintaining Pinax, but I'm not aware of an update. |
Thanks @KatherineMichel for replying. I saw the documentation and It is not yet updated. Could you please suggest to me how can achieve it? I am already using it but now there is requirement to add referral in the query string of a URL |
@surinderdeveloper I think @kevinmickey might have a better idea atm of how to do that, since he suggested it. |
It would be nice to use the referral in the query string of a URL. In order to have backward compatibility, this could be optional. This would have several benefits.
In order to implement, check out: https://github.com/byteweaver/django-referral/blob/master/referral/middleware.py I think the important pinax-referrals code would be moved from a view to middleware.
The text was updated successfully, but these errors were encountered: