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

Implement Asynchronous Processing #10

Open
plunkettscott opened this issue Feb 1, 2021 · 8 comments
Open

Implement Asynchronous Processing #10

plunkettscott opened this issue Feb 1, 2021 · 8 comments

Comments

@plunkettscott
Copy link

Implement Asynchronous Processing as described in the JSON:API Spec recommendation. This exists in the outgoing version of this package but has not been implemented in this version.

@lindyhopchris
Copy link
Contributor

Thanks for opening the issue!

@plunkettscott
Copy link
Author

No problem. If I get some time I can take a look at implementing this as well. Any ideas how you'd like to see something like this implemented?

@lindyhopchris
Copy link
Contributor

Thanks for the offer, it is appreciated but I think I need to do this one. I want it to be in a separate package: laravel-json-api/async as it's an extra feature rather than a core feature.

@lindyhopchris
Copy link
Contributor

Note to self: there was an issue on the old package about investigating improving the implementation by using Job Middleware:
cloudcreativity/laravel-json-api#418 (comment)

@zlodes
Copy link

zlodes commented Apr 19, 2021

@lindyhopchris Hi! Did you have any plans about it?
It's just the last thing that don't let us to upgrade to the brand-new package.

P.S.: It's amazing and much better than the previous one. Good job! Thank you!

@lindyhopchris
Copy link
Contributor

@zlodes thanks for the feedback, it's really good to hear that it's an improvement - I also prefer the new package!

The only thing blocking this is time at the moment. The production APIs I'm working on at the moment don't use the async feature, which is why it hasn't bubbled up to the top of my priorities. It's really difficult for me to estimate at the moment when I might get on to it.

@makowskid
Copy link

Hi! Any update on this topic? Any plans to actually add async processing?

@lindyhopchris
Copy link
Contributor

Yeah it's on the list of things I need to work through for #238 - I made progress on that ticket during the summer but have had to put it down again due to other priorities. Planning to pick back up sometime soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants