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

Migrations are ignored #237

Closed
Vandaahl opened this issue Dec 29, 2020 · 4 comments
Closed

Migrations are ignored #237

Vandaahl opened this issue Dec 29, 2020 · 4 comments

Comments

@Vandaahl
Copy link

I have installed a fresh copy of Sylius Standard Edition version 1.8.7. Then I install the Refund plugin according to the instructions:

composer require sylius/refund-plugin

I accept the recipe and run the migration:

bin/console doctrine:migrations:migrate

The output is as follows:

[OK] Already at the latest version ("Sylius\Bundle\AdminApiBundle\Migrations\Version20170711151342")

My src/Migrations folder is empty. No database changes are made. When I copy the migration files from vendor/sylius/refund-plugin/migrations/* to src/Migrations and run the migration command again, nothing changes because I get the same output as above.

I do not have alot of Symfony or Sylius experience yet so I'm not sure where the problem lies. I think it's a bug though.

@mxkh
Copy link

mxkh commented Dec 29, 2020

I can confirm this issue. Everything was done following the official documentation, got the same issue

@Zales0123
Copy link
Member

Hi! I believe there is a problem in the documentation. When you run just composer require sylius/refund-plugin the 0.10.1 version is installed by default. To install the latest, RC-5 version, you need to run composer require sylius/refund-plugin:v1.0.0-RC.5. It would not be a problem after the stable release :)

Are you willing to open a PR with such a change? Thanks! 🖖

@Vandaahl
Copy link
Author

Vandaahl commented Jan 1, 2021

Thanks Zales0123, I was able to install the plugin correctly. I'm not yet familiar with doing pull requests, so I'll look into it.

@GSadee
Copy link
Member

GSadee commented Mar 16, 2021

Closing as the installation guide has been fixed in #267

@GSadee GSadee closed this as completed Mar 16, 2021
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

No branches or pull requests

4 participants