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

Publish button is not consistantly displaying PDP or homepage #2975

Closed
kieckhafer opened this issue Sep 28, 2017 · 7 comments
Closed

Publish button is not consistantly displaying PDP or homepage #2975

kieckhafer opened this issue Sep 28, 2017 · 7 comments
Assignees

Comments

@kieckhafer
Copy link
Member

Expected behavior

PDP and grid should always have publishing buttons (if appropriate)

Actual Behavior

Publish button does not consistently display on PDP or grid. Sometimes it's there, sometimes it's not, although all other admin functions seem to always work regardless

Steps to Reproduce the Behavior

  • Log in as an Admin.
  • Go to PDP
  • Refresh, do some actions, see the button going in and out

Versions

Node: 6.11.3
NPM: 5.4.2
Meteor Node: 4.8.4
Meteor NPM: 4.6.1
Reaction CLI: 0.12.0
Reaction: 1.5.0
Reaction branch: ek-updateVariantForm
Docker: 17.06.2-ce

basic_reaction_product

@mikemurray
Copy link
Member

Adding to this. While signed in, keep refreshing the homepage. Eventually, you'll get a grey screen in the middle area. The toolbars remain, but the navbar, product grid are gone.

This doesn't appear to happen for a signed out user.

@spencern
Copy link
Contributor

@mikemurray Might be worth trying this with meteor 1.5.2.2 meteor/meteor#9151 which has a fix for the bug where people were getting logged out. Not sure they'd be related, but maybe possible if we just started noticing this

@mikemurray
Copy link
Member

@spencern meteor 1.5.2.2-rc.0 does not solve the issue, so my journey continues.

@mikemurray
Copy link
Member

From what I can tell, part of the issue is the index route has no layout structure or incorrect structure in some cases, causing the grey screen.

@mikemurray
Copy link
Member

Digging deeper into the router. Inside the "ReactionLayout" function the call to Router.Reaction.getPrimaryShopId() sometimes return null where Router.Reaction.getShopId() always returns the shopId.

Not sure why they would be, any ideas @spencern ?

@spencern
Copy link
Contributor

Not sure why that would be either, but let's pair on this.

@mikemurray
Copy link
Member

mikemurray commented Sep 29, 2017

Fixed as part of #2982

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

3 participants