swellstores / horizon

Headless NextJS storefront starter powered by Swell
https://swell-horizon-demo.vercel.app/
MIT License
58 stars 52 forks source link

Menu updates aka whac-a-mole. #48

Closed holden closed 1 year ago

holden commented 1 year ago

Hi.

I'm not sure if this is specific to horizon or not but I'm having a problem with changing settings related to the theme and menus.

Specifically when I edit the navigations, changes I make tend to undo other changes. Eg if I try to add a footer, it removes the header and vice versa. Sometimes when I try to add the header back it removes the footer and the header.

I know both my header and footers work, but I can't seem to get these changes saved at the same times.

Is there some setting I don't have correct setup that would cause this issue?

I made a video:

https://github.com/swellstores/horizon/assets/48241/c4cc116f-0091-45ed-a72e-f326ff54a83a

holden commented 1 year ago

I figured it out, apparently I had "All categories" in the menu which for whatever reason causes a 404 which I found in the console. Once I removed it from the menu it works again.

holden commented 1 year ago

After I had figured it out, they both promptly stopped working again while I was making a few very small changes only within the storefront theme editor, this time with no errors in the console. In this instance I added social icons after which both menus seem to start appearing although I can get them to appear temporarily but even loading the editor can cause both to disappear again.

Is there some way I can debug whatever is going on? It's like there's a small demon inside the storefront editor, I would be terrified to make any changes if this were in production. What could cause this and how can I debug the issue? The are no errors in either the console or when I run it locally in development.

logeshswell commented 1 year ago

@holden Responded to this issue in the below GitHub discussion thread: https://github.com/orgs/swellstores/discussions/271#discussioncomment-6303390

I will keep you posted if this bug is resolved.

logeshswell commented 1 year ago

This issue is fixed and deployed in production. Please check and confirm.