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

Add GoPay gateway #4

Merged
merged 1 commit into from
Nov 23, 2014
Merged

Add GoPay gateway #4

merged 1 commit into from
Nov 23, 2014

Conversation

sergej-koscejev
Copy link
Contributor

My gateway is ready and I'd like to add it to your multiprocessor, if possible. Thanks.

By the way, how do I determine the URL to be used for notifications?

@eileenmcnaughton
Copy link
Owner

I have been hard-adding the vendor folder into the extension too. I know this isn't probably the right way to do it but have been trying to stick to a simple install for now since civicrm + composer are still in early pages.

URL is civicrm/payment/ipn?processor_id=x

where x is the processor id. I think processor_name is also an option but it doesn't distinguish between processors of the same type

eileenmcnaughton added a commit that referenced this pull request Nov 23, 2014
@eileenmcnaughton eileenmcnaughton merged commit 63b73e5 into eileenmcnaughton:master Nov 23, 2014
@eileenmcnaughton
Copy link
Owner

NB - I guess I can re-run composer & do that hard-add myself

@sergej-koscejev
Copy link
Contributor Author

Yes, forgot to mention the Composer thing - didn't want to make this pull request too big.

@sergej-koscejev
Copy link
Contributor Author

Thanks for merging! Would it be possible for you to make a new release of the multiprocessor, by the way? Or is there maybe some easy way to install it from Github? I developed this for a client and it would be nicer if they didn't have to give me access to their server. Or am I asking too much? :)

@eileenmcnaughton
Copy link
Owner

I've tagged it as a release - but I don't know how often the script that publishes newly tagged releases works - or if it always works.

Probably worth you retesting locally at this point because there was a bunch of composer changes (including to paybox which I have in process)

@eileenmcnaughton
Copy link
Owner

closing - I got an email so I assume the release worked :-)

@eileenmcnaughton
Copy link
Owner

oh can't close -it's not an issue :-)

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