Skip to content
This repository has been archived by the owner on Apr 27, 2023. It is now read-only.

feat: configure page paths via options #12

Merged
merged 2 commits into from Jul 12, 2019
Merged

Conversation

notrab
Copy link
Collaborator

@notrab notrab commented Jul 11, 2019

Adds config to change the path to the catalog pages.

@notrab notrab requested a review from ynnoj July 11, 2019 12:19
@vercel
Copy link

vercel bot commented Jul 11, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://gatsby-theme-moltin-git-feat-custom-catalog-paths.moltin.now.sh


return {
plugins: [
{
resolve: `@moltin/gatsby-source-moltin`,
options: { client_id }
options: { client_id: client_id || clientId },
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, we should support both and phase out client_id.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@notrab This should be reflected at gatsby-source-moltin too.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ynnoj Yep! Done, will submit another PR later.


return {
plugins: [
{
resolve: `@moltin/gatsby-source-moltin`,
options: { client_id }
options: { client_id: client_id || clientId },
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@notrab This should be reflected at gatsby-source-moltin too.

@notrab notrab merged commit 2af8492 into master Jul 12, 2019
@notrab notrab deleted the feat/custom-catalog-paths branch July 12, 2019 12:23
notrab added a commit that referenced this pull request Jul 12, 2019
* feat: configure page paths via options

* docs: update README
@ynnoj
Copy link
Owner

ynnoj commented Jul 12, 2019

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ynnoj ynnoj added the released label Jul 12, 2019
notrab added a commit that referenced this pull request Jul 12, 2019
* feat: configure page paths via options

* docs: update README
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants