John Albin Wilkins

Results 117 comments of John Albin Wilkins

This PR is blocked because all the examples need to be updated to use the App Router.

The Pages Router-specific methods in DrupalClient are: * `getResourceCollectionFromContext()`: The App Router equivalent is `getResourceCollection()` * `getResourceFromContext()`: The App Router equivalent is `getResource(type, uuid)` or `getResourceByPath(path)` * `getSearchIndexFromContext()`: The App...

PR #716 adds two new methods: # constructPathFromSegment() This is the refactored `getPathFromContext()`. It returns a Drupal path given an array of Next.js path segments, path prefix and locale. #...

Why can't the existing preview code be extended to show unpublished nodes? Is there a technical reason for adding a completely new route and controller?

Yep! The purpose of the feature is clear and obviously useful. I was asking for clarification about the code used. Please re-read the questions.

How is this different than just setting the base path as part of the base url on `admin/config/services/next/sites/[some-site]/edit`?

FYI, `packages/next-drupal/src/get-menu.ts` is _not_ the method used in `DrupalClient`; it's the deprecated code before `DrupalClient` was introduced in 1.6. The type definition for the `getMenu` method is in `src/client.ts`. With...

I'm wondering if we should flip this around. Instead of having a `useDefaultResourceTypeEntry` optimization that 90% of people will want. What if we add a config to opt-in to querying...

2.0.0-beta.0 will introduce a new `NextDrupal` class that has a `useDefaultEndpoints` option that defaults to `true`. When `useDefaultEndpoints` is `true`, the `NextDrupal` object will programmatically-create the JSON:API endpoint instead of...

The docs need to be updated to reflect that the patch is no longer needed.