americanhandelsociety / americanhandelsociety-members

0 stars 0 forks source link

Issue 132 #138

Closed ecedmondson closed 1 year ago

ecedmondson commented 1 year ago

I was able to test this locally. The new template is rendering with the css as well as the payments confirmation. Copy is, I guess, up to you, but I gave it a shot.

I thought perhaps this view should not just be authentication protected but referral protected (i.e. only load with referer headers from paypal) but I don't know if 1) paypal supports this or 2) if you want this. Either way, I think that part would be dependent on your work. As it stands, this simply can be navigated to by discovering the URL.

I also thought that an outstanding piece is providing the URL to paypal for referral but I think I would need paypal access to do that, and don't have it (nor should I, I don't think).

ecedmondson commented 1 year ago

I think making it authentication-protected view will suffice. I can do that in my forthcoming PR.

This part should already be done, unless I misunderstood the requirement.