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 @products and @categories table groups for database dumps #286

Closed
wants to merge 2 commits into from

Conversation

tegansnyder
Copy link
Contributor

@paales GIST here gets the credit: https://gist.github.com/paales/8229883 but I think this should be a core feature for database dumps :)

@cmuench
Copy link
Member

cmuench commented Jan 8, 2014

Thank for this. What's up with bundle tables?
I'm not sure if the table list is really complete.

@paales
Copy link

paales commented Jan 8, 2014

I have based it loosely on https://gist.github.com/philwinkle/0a1725635204e07a69a5
Discussion can be found here: https://twitter.com/paales/status/418882572647952384

What's up with bundle tables?

Not sure if I understand what you mean, all bundle tables are excluded as far as I can see.

@cmuench
Copy link
Member

cmuench commented Jan 8, 2014

Yes, that's the point. Product data is much more. If you use i.e. the @sales group you don't wanne have any data of orders. If i add @Product i don't wanne have any product. If a bundle product exists we should exclude the bundle data also.
Before we merge this we should check if the table list is really complete for all standard product types.

@paales
Copy link

paales commented Jan 8, 2014

Suggestions:

  • downloadable_
  • product_alert
  • rating?
  • review?
  • tag
  • wishlist

Bonus Questions: How does it handle url rewrites if the products aren't available? I believe the URL rewrites have a foreign key to the product.

@cmuench
Copy link
Member

cmuench commented Feb 18, 2014

I close this pull request. The task is much more complex. We need another solution to support this.

@cmuench cmuench closed this Feb 18, 2014
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

3 participants