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

[PDP] Upgrade from 0.16.0 product #1488

Closed
aaronjudd opened this issue Oct 11, 2016 · 7 comments
Closed

[PDP] Upgrade from 0.16.0 product #1488

aaronjudd opened this issue Oct 11, 2016 · 7 comments
Assignees
Milestone

Comments

@aaronjudd
Copy link
Contributor

aaronjudd commented Oct 11, 2016

Upgrading from a shop with 0.16.0 product causes some unexpected visibility issues.

Expected behavior

That a product from a pre 0.17.0 release should smoothly migrate without issue to a 0.17.0 product with revisions.

Actual Behavior

Product grids display "No products found".

Steps to Reproduce the Behavior

something like this...

reaction init -b master
reaction
git checkout development
@brent-hoover
Copy link
Collaborator

I am pretty sure it has something to do with this:

Possibly just a migration to add this field and set it to false would do the trick?

@brent-hoover
Copy link
Collaborator

Or am I just stating the suuuper obvious here

@aaronjudd
Copy link
Contributor Author

or just isDeleted: {$in: [null, false]},?

@brent-hoover
Copy link
Collaborator

Does that work if the field doesn't exist? If so, yeah that's way better. I'm testing now.

@aaronjudd
Copy link
Contributor Author

aaronjudd commented Oct 11, 2016

Product view would need an update as well,

selector.isDeleted = false;
could be

selector.isDeleted = {$in: [null, false]};

@brent-hoover
Copy link
Collaborator

So I also noticed when on this migrate path I get the old PDP page rather than the new version. Is that intentional or do I need to turn revisions on somewhere?

@brent-hoover
Copy link
Collaborator

I have revision control enabled but get the old PDP page so no revision control.

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

2 participants