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

Can't create any overview or landing page #17

Closed
vasiok32k opened this issue Nov 12, 2020 · 14 comments
Closed

Can't create any overview or landing page #17

vasiok32k opened this issue Nov 12, 2020 · 14 comments
Assignees

Comments

@vasiok32k
Copy link
Contributor

vasiok32k commented Nov 12, 2020

Hello
After installing the module but was found issues during creating landing page and overview page, after click on button Save or Save and Continue Edit, page will be frozen and got error in the console related of Page Builder
image
I partly investigated issue and it seems like it's caused by wrong component structure, a component within a component that can't handle a component. As a solution could be that move Page Builder instances behind the button that will need to click a button to open the Page Builder - need to add that config line in appropriate places where adding Page Builder instances:

<item name="pagebuilder_button" xsi:type="boolean">true</item>

It will solve the problem but will lead to another PHP error like this:
image
Could you please fix that issues or at least tell me please how to could be resolved?

@edwinljacobs
Copy link
Contributor

Hi,

I assume you are on enterprise? Which version exactly? To be honest i'm not sure why this is an issue. I would have to investigate that, sadly I cannot do that today. I think I will be able to provide you with an update on Wednesday. By the way, I assume you are using this in combination with tweakwise?

I did take a quick look to find similar issues, maybe this helps in the meantime? mirasvit/module-blog#162

With kind regards

@edwinljacobs edwinljacobs self-assigned this Nov 12, 2020
@vasiok32k
Copy link
Contributor Author

Version is 2.3.5-p2

@vasiok32k
Copy link
Contributor Author

vasiok32k commented Nov 12, 2020

I will really appreciate you if you will help me to fix it (about that link - I already looked that resource before and it not helped me unfortunately)

@edwinljacobs
Copy link
Contributor

Hi,

I took a quick look, It seems that we have been sloppy with our method definition in
Emico\AttributeLanding\Model\IrlRewriteService::generateRewrite();

diff --git a/src/Model/UrlRewriteService.php b/src/Model/UrlRewriteService.php
index 0d0650f..b22c72c 100644
--- a/src/Model/UrlRewriteService.php
+++ b/src/Model/UrlRewriteService.php
@@ -109,10 +109,10 @@ class UrlRewriteService
     }
 
     /**
-     * @param LandingPage $page
+     * @param UrlRewriteGeneratorInterface $page
      * @param string|null $suffix
      */
-    public function generateRewrite(LandingPage $page, string $suffix = null)
+    public function generateRewrite(UrlRewriteGeneratorInterface $page, string $suffix = null)
     {
         $urlRewritesToPersist = [];

You could try to apply this patch and check if that works (I haven't tested it). The overview page functionality is seldom used and perhaps under developed.... Ill look into this with more detail wednesday.

With kind regards

@edwinljacobs
Copy link
Contributor

I made a note for testcases as this kind of stuff should not happen

@vasiok32k
Copy link
Contributor Author

Thanks seems like it helps for overview page, but landing page still can't be save, getting that error
image

@vasiok32k
Copy link
Contributor Author

Will wait Wednesday

edwinljacobs pushed a commit that referenced this issue Nov 18, 2020
…, fixed type error in url rewrite generator
@edwinljacobs
Copy link
Contributor

Hi,

I created release https://github.com/EmicoEcommerce/Magento2AttributeLanding/releases/tag/v1.1.3 for this issue.
It should be fixed now. One thing though, I assume you are using this module in combination with
https://github.com/EmicoEcommerce/Magento2Tweakwise
https://github.com/EmicoEcommerce/Magento2TweakwiseExport
https://github.com/EmicoEcommerce/Magento2AttributeLandingTweakwise

The reason I'm asking is that if this is not the case then it wont work. The mechanism that should apply the filters in the field Filter attributes (in the form above) to the product list is not implemented in native magento. See

//@todo Magento native implementation for applying filters

@vasiok32k
Copy link
Contributor Author

Hi
Thanks a lot for you time, seems like it fixed and works well.
I also created PR about error related of Page Builder, please review it and merge if it acceptable for you (because moving Page Builder instances behind the buttons really fixing that problem)
#18

@edwinljacobs
Copy link
Contributor

Thanks for taking the time to make a PR! it has been released in https://github.com/EmicoEcommerce/Magento2AttributeLanding/releases/tag/v1.1.4

I hope this solves your issues, good luck!

@vasiok32k
Copy link
Contributor Author

Yes, it solved my issues, thanks for your activities and help.

@vasiok32k
Copy link
Contributor Author

Tell me please how to became a contributor to be in this list?
image

@edwinljacobs
Copy link
Contributor

Yeah weird, that doesn't work, perhaps https://stackoverflow.com/questions/40644933/not-showing-as-a-contributor-on-github helps, the accepted answer is not applicable but maybe something in your email settings (the second answer)?

@vasiok32k
Copy link
Contributor Author

Hi
I contacted with support of the Github and we resolved the problem, thanks.

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