Skip to content

Conversation

@hantastic
Copy link

This code adds the ability to pass in an optional channel name to override the one specified in the env. This will mean getAgilityPaths and getAgilityPageProps can access more than one channel in a single site / app

Copy link
Member

@joelvarty joelvarty left a comment

Choose a reason for hiding this comment

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

This looks awesome! Nice work! @joshua-isaac - can you take a look as well, and everything is good, I can bump the version number and publish to NPM?

@joelvarty joelvarty requested a review from joshua-isaac April 12, 2022 13:23
@joelvarty joelvarty self-assigned this Apr 12, 2022
@joelvarty joelvarty added the enhancement New feature or request label Apr 12, 2022
fix agilityPaths typing
@joshua-isaac
Copy link
Contributor

@joelvarty fixed some typing here to get agilityPaths to work. would you mind checkin it out when you get a chance?

@joshua-isaac joshua-isaac requested a review from joelvarty April 13, 2022 21:23
@joshua-isaac
Copy link
Contributor

we may also want to rename the prop to just sitemap or something

@hantastic
Copy link
Author

I've changed this to sitemapName because sitemap was being used elsewhere in the file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants