-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Attributes] Add attributes overview #18003
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
I like this overview a lot 😎 cc @fabpot |
Thanks for the PR 👍! I made some suggestions for missing attributes with the refs to add! |
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.
Thanks, looks good!
562c8ff
to
f627de9
Compare
f627de9
to
46981e7
Compare
Thank for the review @wouterj, that's updated 🙂 |
Thanks a lot for this work, this help to discover symfony features in one page. (I did not find any forgotten property from codebase, good work !) |
As always, great addition for the Symfony documentation. Mayny thanks ❤️ |
We would love to see the follow up PRs against |
…daubois) This PR was merged into the 6.2 branch. Discussion ---------- [Attributes] Add attributes overview for 6.2 Waiting upmerge of #18003, but here is the 6.2 followup 👍 It seems there will be some conflicts during upmerge. I'm keeping an eye on this and will update this PR accordingly I'm on 6.3 as well! Commits ------- f25d340 [Attributes] Add attributes overview for 6.2
…daubois) This PR was merged into the 6.3 branch. Discussion ---------- [Attributes] Add attributes overview for 6.3 Followup of #18003 and and waiting merge of #18014 Again, I'm keeping an eye on upmerge and will rebase accordingly 🙂 Commits ------- 095824b [Attributes] Add attributes overview for 6.3
Fixes #17981. Once merged, I'll full-fill the page for 6.2 and 6.3 👍