Open bph opened 1 year ago
I am woring on this. Planned date for the draft—November 20th.
@OlaIola Just checking in if there are any blockers on your progress I could help remove?
The article draft: https://docs.google.com/document/d/1Dre7ykgmfqH0IuyBQJBhKiHykQshyJTHz0SslqiJ2Jc/edit?usp=sharing @bph @zzap
@OlaIola, thank you so much for working on this topic. It is an extremely important one, as I see many bad practices resulting from not understanding what happens when a query is processed.
I reviewed the draft. At first, I started leaving comments directly in the document, but then I noticed the same issues being repeated, so I'll give general feedback here.
The draft does not feel ready for review. The examples and descriptions of problems are unclear. I don't understand the situations you are explaining, and then I'm not sure if the code example shows good or bad practice. I would suggest revising each problem and trying to answer the following questions:
You're listing quite a few situations/problems. I'd rather choose 3-5 of the most performance-impactful ones and go into detail with them than have 21 unclear ones. It is OK to state that the article will not cover all situations in detail and just list the rest as the ones to pay attention to and thoroughly test when using.
Please let me know if you want to discuss this further. Thank you :heart:
The second version is waiting to be reviewed. The same file: https://docs.google.com/document/d/1Dre7ykgmfqH0IuyBQJBhKiHykQshyJTHz0SslqiJ2Jc/edit?usp=sharing
Discussed in https://github.com/WordPress/developer-blog-content/discussions/27