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

Added PayPal requriement #12

Merged
merged 1 commit into from
Feb 17, 2016
Merged

Conversation

pivulic
Copy link
Contributor

@pivulic pivulic commented Feb 17, 2016

Problem

During our deployment process, where the database is created, it crashes in https://github.com/Adyen/adyen-magento2/blob/1.1.1/Setup/UpgradeSchema.php#L90 due to:

Module 'Adyen_Payment':
Installing schema.. Upgrading schema.. 

  [Zend_Db_Statement_Exception]                                                                                                                                  
  SQLSTATE[42S02]: Base table or view not found: 1146 Table 'project_m2.paypal_billing_agreement' doesn't exist, query was: DESCRIBE `paypal_billing_agreement`  

  [PDOException]                                                                                                 
  SQLSTATE[42S02]: Base table or view not found: 1146 Table 'project_m2.paypal_billing_agreement' doesn't exist

Expected Result

  1. Adyen should depend on PayPal to be installed.
  2. A new version in https://packagist.org/packages/adyen/module-payment

@rikterbeek
Copy link
Contributor

Hi Pablo,

Thanks, yes that is correct. I will merge your pull request.

Regards,
Rik
Adyen

rikterbeek added a commit that referenced this pull request Feb 17, 2016
@rikterbeek rikterbeek merged commit 7829840 into Adyen:develop Feb 17, 2016
@rikterbeek
Copy link
Contributor

Hi Pablo,

Will be in the next release of the module somewhere next week. For now you could use the develop branch or fist install magento2 without adyen module install and do it afterwards.

Regards,
Rik
Adyen

@pivulic
Copy link
Contributor Author

pivulic commented Feb 17, 2016

Thanks Rik, we'll use the develop branch meanwhile.

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.

2 participants