Webiny-js: Fix links included in the initial Page Builder pages

Created on 29 Jan 2020  ·  7Comments  ·  Source: webiny/webiny-js

Initial Page Builder pages, that come out-of-the box with every Webiny installation, contain links to some of the docs.webiny.com pages.

But, a few links on the For Developer and Get Started pages point to non-existing pages. So we need to fix this (and we probably need to check all of the links on these initial pages).

And additionally, for some reason, all of the links in these pages are prefixed with the current site's domain, so basically all links are broken.

The fix for that should probably land somewhere in the packages/app-page-builder/src/render/plugins/slate/link/index.tsx file.

bug good-first-issue

Most helpful comment

Sure, I'm very easygoing with the issue.

If this is critical I may always speed it up 😄. I'm coming with questions / feedback soon.

All 7 comments

I'd like to take this on ☄

Be my guest @Fsalker :)

I will provide a few instructions quickly @Fsalker . Hold on :)

Added a short document that says a few words about the initial data and files that are utilized in the Page Builder process (all located in a single ZIP file).

https://github.com/webiny/webiny-js/blob/master/packages/api-page-builder/README.md

The links we're talking about in this issue are all located in the initial pages that are created in the Page Builder installation process, and these pages can be found in the mentioned ZIP file.

So you will have to unzip that, correct the broken links in the data/pagesData.json, create a new ZIP and commit it.

Once you're done let me know, and I'll test it out.

And of course if you have more questions, feel free to ask, I'm here.

Thanks!

Sure, I'm very easygoing with the issue.

If this is critical I may always speed it up 😄. I'm coming with questions / feedback soon.

Thanks!

@Fsalker, it's not critical, take as much time as you need!

Thank you! 🍻

Was this page helpful?
0 / 5 - 0 ratings