Gatsby: Change prop name "pathContext" to "pageContext"
This when creating pages you can add a context key. All data passed there is passed into pages as pathContext. I think pageContext is a more intuitive name as its context for the "page" which is the main object not "path", an attribute of the page.
Since we're at 1.0, we can just pass in both and then remove pathContext at the next major release.
Any thoughts?
KyleAMathews
All 4 comments
pageContext is less confusing IMO.
Maybe gatsby can throw warnings when building with deprecated pathContext?
Then, it wouldn't be a surprise in 2.0.
Vagr9K
on 10 Jul 2017
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
![stale[bot] picture](https://avatars3.githubusercontent.com/in/1724?v=4&s=40)
stale[bot]
on 6 Jan 2018
Totally agree, pathContext is confusing. pageContext will definitly fit better!
monsieurnebo
on 9 Jan 2018
Merged into v2 branch. Going to close this.
calcsam
on 26 Jan 2018
Related issues
andykais
路
3Comments
hobochild
路
3Comments
ghost
路
3Comments
3CordGuy
路
3Comments
benstr
路
3Comments
Most helpful comment
Merged into v2 branch. Going to close this.