-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[JsonPath] Add the component #20837
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
Comments
I just leaned about the component at Symfony Live, I'm going to look into it and if I feel confident enough I'll try to do the doc 😊 |
Thank you for your interest on this @RisingSunLight42. Feel free to ping me on the Symfony Slack if you need any information/support, I'll be happy to help 🙂 |
Hello @RisingSunLight42, did you have time to check this issue? The component will be officially released at the end of May with Symfony 7.3, it would be nice to have the documentation ready (or almost ready) in the next few weeks 🙂 |
Hi @alexandre-daubois! I started looking into it, as a warmup into writing doc for Symfony I was taking less time consuming issues before, but I'll work on it during the next weeks. I'll also open a draft PR so everyone can see how it's going. |
Great news, thank you! |
This should also cover the |
Noted, thank you! |
Hi @RisingSunLight42, Symfony 7.3 is just around the corner as well as the first release of the component. Did you have time to start the work? 🙂 If you don't, please tell me if I can take over. Thanks! |
I started a rough thinking of it (nothing written, just plan and how organizing everything). I plan on opening a draft PR in the next days (probably today or tomorrow) with the basic structure and wording. I'm fine with you taking over if you want, I got unexpectedly busy the last weeks so everything did not flow as I planned 😕. For the plan, I was planning to do something similar to the YAML Component doc, obviously with adaptations for the JsonPath component. So something to explain why using it instead of |
If you have a draft PR, let's not lose your work and iterate on it. Docs core team and contributors will be able to give you feedback and get this to the finish line 👍 |
We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.
To fix this issue, please create a PR against the 7.3 branch in the symfony-docs repository.
Thank you! 😃
The text was updated successfully, but these errors were encountered: