-
Notifications
You must be signed in to change notification settings - Fork 115
Examining the state of your PHP code #2379
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
Lesson Development Checklist
|
@agiljulio, here is the script and audio file. I don't have a pre-existing video, because this will be a brand new lesson, so you'll have to create the visuals yourself, in the same style as the previous lesson. Are you comfortable to take on this lesson video next? |
Hi @jonathanbossenger thank you for sending the script and audio file. Noted it, I will adjust the style to the same as the previous video. Yes, I feel comfortable taking on this lesson for the next video, If I have any questions, I will ask here 🙏 |
Thank you @agiljulio I look forward to reviewing the next video. |
Hi @jonathanbossenger the video has been edited and is ready for review. Please feel free to provide any feedback, thank you 🙏 |
Thank you @agiljulio I will do my best to review it today or tomorrow. |
Hey @agiljulio sorry for the delay here. Great work on the video. I'm going to open this for the training team to review, and then get started on the scripts and voice recordings for the next two videos. I'd like to try and get these to you by the end of this week. |
Nice clear overview of debugging @jonathanbossenger I have one piece of feedback, i'm not sure if I missed it.
|
Thanks @digitalchild This configuration was covered in the previous lesson in this module on enabling the debugging options. Do you think it would be beneficial to refer to this in the spot you mention? |
@jonathanbossenger Yes I think if you mention its in the previous lesson that should be enough. The reason I ask is because you have the wp-config open in the IDE :) |
Aha, that's a solid point. I didn't actually create the video, Agil and I are collaborating on this one. I'm planning my own review later, so I'll keep an eye out for this. |
@digitalchild at 0:28 in this lesson, it directly references the "The built in WordPress debugging options" lesson, which contains the relevant settings to enable debug.log (here is the relevant section in that lesson). As this is the next lesson in the learning pathway after that one, would you agree this is sufficient reference to that lesson, and the settings therein? |
Hi @agiljulio As always, a great video. I do have a few change requests. Mostly they are adding some additional pauses in the voice, to allow viewers to take in what's on the screen.
Thanks in advance. |
Hi @jonathanbossenger , thank you for your feedback. It was very detailed and will help me maintain the quality of the video. Here is the updated version. |
Thanks @agiljulio it's looking great. I did pick up one additional small bug. At 2:15 when you update the code to wrap So instead of
It should just be
Apologies if this wasn't clear in the script, I'll make sure to make these examples clearer going forward. |
No problem @jonathanbossenger . Here is another update. Thank you for clarifying 🙏 |
No problem, @agiljulio. Thank you for always being open to updates and changes. I will work on getting this video lesson published this week and share the next two scripts with you. |
My pleasure @jonathanbossenger 🙏 I look forward to receiving the next script. |
Uh oh!
There was an error while loading. Please reload this page.
Details
Learning Objectives
Related Resources and Other Notes
Automation Code
//lesson
The text was updated successfully, but these errors were encountered: