-
Notifications
You must be signed in to change notification settings - Fork 8
Add support for Magento 2 #34
Comments
McBrides on it. |
Get me a shout if you want to chat about the approach. |
@brideo What's the status on this? I see you've made a lot of progress on your fork. |
Yeah I haven't forgot about this, feel free to pick it up I'm just a bit flat out. The PutCommand still needs some work. I'm not entirely happy with the stripping implementation as it's just an array of table names, n98 pretty much do the same just with a yaml file. https://github.com/brideo/magedbm/tree/magento2-support/src/Meanbee/Magedbm The factory logic could also be improved but MVP bro. |
I can schedule to smash some stuff out next week, integration tests would be pretty cool. |
@brideo Yo, where we at? Got anything worth salvaging if we're looking into this again? |
@bobbyshaw hey mate, sorry I haven't made any progress since my last comment. https://github.com/brideo/magedbm/commits/magento2-support I think Magerun is still a dependency. I got pretty far so I would use my fork, do you want me to submit a PR or will you just scrap it and start again? |
I'd definitely look over it and try and use it ;) Submit a PR and I'll get it accepted into a new branch for us to play with. |
I'll just leave this here 😄
The text was updated successfully, but these errors were encountered: