Skip to content

Latest commit

 

History

History
115 lines (72 loc) · 4.5 KB

CONTRIBUTING.md

File metadata and controls

115 lines (72 loc) · 4.5 KB

Contributing to @prezly/theme-kit-nextjs

First off, thanks for taking the time to contribute! 🎉
All work on @prezly/theme-kit-nextjs happens directly on GitHub. Both core team members from Prezly and external contributors send pull requests which go through the same review process.

Development

Assuming you have cloned the repo and installed dependencies on your machine, the very next step is to develop the package.

Since theme-kit provides a collection of base functions, hooks, and utilities that work with Prezly themes under the hood, in order to make changes and test things out on your side, you're gonna need to keep a theme running in parallel with the theme-kit.

So if you haven't already, go ahead and follow the instructions on Bea theme to get it up and running.

As you can already see in the theme's package.json file, we're using @prezly/theme-kit-nextjs package. So we need to make sure that our theme is using our local version of theme-kit and not the one's in the theme's node_modules folder. How do we achieve that?

Option 1 - Using yalc (Preferred way)

There's this awesome package called yalc, which serves the purpose for our use case! So let's install it:

npm i yalc -g

Or if you're using yarn:

yarn global add yalc

The yalc package maintains its own local "store", at ~/.yalc. Publish a package with yalc, and a full copy of the package is copied to the store. Install a package from the yalc store, and the project will install that copy much like it would install a package from an external registry. Update a package published to the yalc store, and the update is now available in the dependent projects; you can even publish and automatically update dependent projects with a single command.

Go back to theme-kit's project root folder and run:

yalc publish

This will publish the package in the yalc's local store, then go back to your theme's project root, and run the following commands:

npm uninstall -S @prezly/theme-kit-nextjs
yalc add @prezly/theme-kit-nextjs
npm install

What did we do? We simply removed the theme's @prezly/theme-kit-nextjs package in node_modules and told our theme to use the local version of the package on our file system!

Now the fun part begins, go ahead and hack the package (Make changes) :) Once you're done, simply run the following in theme-kit's project root folder:

yalc publish --push

It acts like you have released a new version of your package that your theme can use.

If the changes are not automatically picked up by theme's dev server, restart the theme's dev server.

Option 2 - Using npm link

In the theme-kit's project root folder run the following command:

npm link

Then go back to theme's project root and run:

npm link @prezly/theme-kit-nextjs

This will create a symbolic link between local version of the package and theme. Checkout Common Errors section in case you have reached unexpected errors.

Common Errors

  1. In case you reach this error in theme and you are unable to run project:

    ⚠ Hooks can only be called inside the body of a function component.

    It means there are two different versions of react running, so in order to fix it we need to make sure both theme and theme-kit are using the same react versions/instance. Assuming that your folder structure looks like this:

    root/
├─ theme-kit-nextjs/
├─ theme-nextjs-bea/

    To fix the error you need to go back to theme-kit root folder and run:

    npm link ../theme-nextjs-bea/node_modules/react
npm link ../theme-nextjs-bea/node_modules/react-dom

  2. Changes are not picked up by yalc after changing deps:
    Try deleting .next folder in theme and restart the dev server.

Bugs

We use GitHub issues to track issues for this repository. Before filing a new issue, try to make sure your problem doesn't already exist.

The best way to get your bug fixed is to provide a reduced test case. Please provide a public repository with a runnable example.

License

By contributing to @prezly/theme-kit-nextjs, you agree that your contributions will be licensed under its MIT license.