-
Notifications
You must be signed in to change notification settings - Fork 203
feat: Finalize 2.x Documentation #804
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…panded for Pages router
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
* Add pages router examples to building pages entry. * Add todos about possible changes to on-demand revalidation section.
Fixes #796 --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/chapter-three/next-drupal/issues/796?shareId=XXXX-XXXX-XXXX-XXXX).
www/content/docs/api/README.mdx
Outdated
| A page with all "Article" nodes. | ||
|
|
||
| ```jsx | ||
| import { DrupalClient } from "next-drupal" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@backlineint hi! Should this example now use the new NextDrupal client mentioned above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch - updated
This pull request is for: (mark with an "x")
examples/*modules/nextpackages/next-drupalstarters/basic-starterstarters/graphql-starterstarters/pages-starterGitHub Issue: #
Please add a link to the GitHub issue
where this problem is discussed.
Code changes need test coverage. If you don't know
how to make tests, check this box to ask for help.
Describe your changes
Final revisions to 2.x docs.