Skip to content

Payment gateway core functionality#1

Open
conradsp wants to merge 10 commits intoopenMF:payment-gateway-core-functionalityfrom
conradsp:payment-gateway-core-functionality
Open

Payment gateway core functionality#1
conradsp wants to merge 10 commits intoopenMF:payment-gateway-core-functionalityfrom
conradsp:payment-gateway-core-functionality

Conversation

@conradsp
Copy link

@conradsp conradsp commented Oct 2, 2018

No description provided.

@ippezshelby
Copy link

@conradsp , @rahulg963 Database migration scripts not found in the resource folder hence getting following errors

  1. Table 'payment-gateway.vendor' doesn't exist
  2. Table 'payment-gateway.transaction_data' doesn't exist

Am trying to test it but can not proceed

@conradsp
Copy link
Author

conradsp commented Apr 25, 2019 via email

@ippezshelby
Copy link

@conradsp
Copy link
Author

conradsp commented Apr 25, 2019 via email

@ippezshelby
Copy link

What is the difference between these two repos? https://github.com/openMF/payment-gateway and https://github.com/openMF/payment-hub. Are they the same, if so then one should be deleted not to be confusing. Thanks

@conradsp
Copy link
Author

conradsp commented Apr 25, 2019 via email

@ippezshelby
Copy link

Okay so we use use the payment-hub repo for testing. Could you please update the README to include how to build and deploy it.

@conradsp
Copy link
Author

conradsp commented Apr 25, 2019 via email

@rahulg963 rahulg963 requested a review from Sanyam96 April 25, 2019 18:01
@rahulg963
Copy link
Member

Hi Steve,
I really appreciate your efforts of doing the code refactoring.

@marta
Copy link

marta commented Apr 25, 2019

@conradsp No, I could not. BTW I think you meant to @ someone else.

Vendor vendor = vendorRepository.getVendorById(msgObj.getPaymentChannelId());

switch (vendor.getVendorName().toLowerCase()) {
case "beyonic":
Copy link
Member

Choose a reason for hiding this comment

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

We can create an Enum class containing all the vendor names at a single place.


private String description;

private String callback_url;
Copy link
Member

Choose a reason for hiding this comment

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

We can use either snake casing or camel casing,

spring.datasource.url = jdbc:mysql://localhost:3306/payment-gateway
spring.datasource.username = root
spring.datasource.password = mysql
spring.jpa.hibernate.ddl-auto = update
Copy link
Member

Choose a reason for hiding this comment

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

we can set this value to validate.

@ippezshelby
Copy link

@conradsp , i want to take up the task of updating the README for building the payment-hub.
when i run "mvn clean install" from the sources directory, it downloads the dependency files but fails at some point with the error below:-
_

PS D:\Projects\Incubations\payment-hub\sources> mvn clean install
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] parent
[INFO] payment-hub
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building parent 1.0.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ parent ---
[INFO]
[INFO] --- maven-install-plugin:2.4:install (default-install) @ parent ---
[INFO] Installing D:\Projects\Incubations\payment-hub\sources\pom.xml to C:\Users\Swift3 Technologies.m2\repository\hu\dpc\paymenthub\parent\1.0.0-SNAPSHOT\parent-1.0.0-SNAPSHOT.pom
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building payment-hub 1.0.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[WARNING] The POM for com.interop.ilp.conditions:interop-ilp-conditions:jar:1.3 is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] parent ............................................. SUCCESS [ 0.509 s]
[INFO] payment-hub ........................................ FAILURE [ 1.766 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.681 s
[INFO] Finished at: 2019-04-26T14:39:28+03:00
[INFO] Final Memory: 25M/224M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project payment-hub: Could not resolve dependencies for project hu.dpc.paymenthub:payment-hub:jar:1.0.0-SNAPSHOT: Failure to find com.interop.ilp.conditions:interop-ilp-conditions:jar:1.3 in https://mymavenrepo.com/repo/7l0NBzlPJ5KFILqnwWhy/ was cached in the local repository, resolution will not be reattempted until the update interval of camel-custom-snapshots has elapsed or updates are forced -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR] mvn -rf :payment-hub
PS D:\Projects\Incubations\payment-hub\sources>

_

@conradsp
Copy link
Author

conradsp commented Apr 26, 2019 via email

@ippezshelby
Copy link

@conradsp , it worked well, thanks. Can't we eliminate CONDITIONS EVALUATION REPORT
where there are lots of Positive matches , Negative matches, Exclusions, Unconditional classes lists when running? From here how do i test the program?

Thanks
Regards

@ippezshelby
Copy link

@conradsp Just quickly, are we going to merge all the work done for Payment Gateway into the Payment Hub since it has a use cases already done like Beyonic integration implemented already. I think @edcable we need a webinar session to clear some confussion some of us may already be in. We seem to be having some projects within OpenMF repo with similar functionalities.
If i can mention afew:-

  1. Payment Gateway and Payment Hub
  2. Mobile Wallet, PiexiePay and Mifos Mobile

Thanks

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.

5 participants