Skip to content
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

Add getRelated method. #71

Open
1 of 2 tasks
Tracked by #529 ...
Decipher opened this issue May 30, 2021 · 4 comments
Open
1 of 2 tasks
Tracked by #529 ...

Add getRelated method. #71

Decipher opened this issue May 30, 2021 · 4 comments
Labels
enhancement New feature or request module: druxt Related to the `druxt` module
Milestone

Comments

@Decipher
Copy link
Member

Decipher commented May 30, 2021

Is your feature request related to a problem? Please describe.
There's no simple way to load in all related entities in Druxt.

Describe the solution you'd like

  • Add a getRelated method to the DruxtClient
  • Add a getRelated method to the DruxtStore

Use the JSON:API related endpoints.

Describe alternatives you've considered
N/A

Additional context
https://demo-api.druxtjs.org/en/jsonapi/node/article/e4ced3c1-e14b-48f1-8c0a-c909e808de68/field_tags

Decipher added a commit that referenced this issue Jul 21, 2021
Decipher added a commit that referenced this issue Jul 21, 2021
Decipher added a commit that referenced this issue Jul 21, 2021
@Decipher
Copy link
Member Author

Above commits are incorrectly tagged :|

@Decipher Decipher added the enhancement New feature or request label Aug 14, 2021
@Decipher
Copy link
Member Author

Decipher commented Aug 14, 2021

@d34dman Have you implemented a similar method with your drupal-js-sdk?

@d34dman
Copy link
Contributor

d34dman commented Aug 17, 2021

@Decipher not yet. And to be honest, didn't even plan this far yet.

Atm, am working on an architectural overhaul after some user feedback and difficulty in using the project. (Especially around state and session management).

@Decipher
Copy link
Member Author

No stress, just wanted to make sure not to duplicate effort.

@Decipher Decipher added the module: druxt Related to the `druxt` module label Aug 27, 2021
Decipher added a commit that referenced this issue Sep 10, 2021
Decipher added a commit that referenced this issue Sep 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
Decipher added a commit that referenced this issue Oct 10, 2021
* feat(#71): add getRelated() method

* chore(#71): add example

* chore(#71): update test coverage
@Decipher Decipher added this to the 1.0.0 milestone May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module: druxt Related to the `druxt` module
Projects
None yet
Development

No branches or pull requests

2 participants