-
Notifications
You must be signed in to change notification settings - Fork 7
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
New developer focused workflow article about how the login and registration works in WordPress. #165
Comments
Rajinsharwar The topic was approved in today's Editorial Group Meeting. |
Hi @bph, great to hear that! Looking forward to get this assigned so that I can include more code references, and proof-read the doc. |
Hi @bph, I just finished adding more code examples, and more proof-reads. Requesting a review on this: 🙂 https://docs.google.com/document/d/17N4uVctGa1pk8B2AcqY7KGGlY6yVV9azn85TsGJGALA/edit?usp=sharing |
On it! Before Monday in the US |
@Rajinsharwar to give reveiwers access for comments, could you please make the Google Doc accessible for comments via link? |
Done, updated the access @bph :) |
@Rajinsharwar I took a first look but need to take a break now. I will get back to it sometime tomorrow. Sorry. maybe @marybaum or @justintadlock are available for a review. |
Milana @zzap has volunteered to do a second review on your post. |
Great! Will be waiting for his feedback on this :) |
Hello @Rajinsharwar , I have reviewed your draft and left quite a few comments. First of all, thank you very much for taking the time to write this article. It is an important topic and you certainly spent a lot of time working on it. The majority of code examples need Some code examples are missing internationalisation functions, and some only textdomain. That as well is expected from all code examples living on wp.org. One callback, however, is causing Fatal error. I didn't go into detailed debugging, but maybe some hint would be that you are passing 5 arguments but not actually using them. I don't know how much experience you have in writing software documentation, but I have a feeling that you are going the extra mile to prevent content from being dry or repeated. This is okay, but be careful not to lose the point and clarity while doing so. I commented on a few places which crossed this line and became unclear and difficult to understand or simply too novelistic. With software documentation, be it tutorial or code reference, it's better to be dry but clear than to lose the point in an attempt to be more interesting. Thank you, again, so much for your time and commitment. I'm looking forward to seeing this article published. |
@Rajinsharwar I was wondering on the progress of this article. Are there any blockers I can remove? |
Following the editorial meeting, following up on this article see if any help is needed to move this forward. Thanks to Rajen and all the reviewers for all your time and effort on this so far. |
Hi @bph @abhansnuk, hope you're doing great! Sorry for the delay, I have fixed the issues commented on by @zzap 😃. I hope this is ready for a second review! Let me know if I missed any step! Thanks to @zzap as well for the detailed feedback, yep, I am new to software documentation, so I will of course make sure to follow your suggestions in future cases. Thanks again, and have a great day! 🙂 |
@abhansnuk would you have time to give this a 2nd review? Google Doc |
Hello, I have done the second review and checked in following along on the two items I raised. This is ready. It needs an excerpt adding, contributors names, and any tags etc. Great team work everyone! Can’t wait to see this live. Hope you will join us for another post soon Rajin. |
@Rajinsharwar seems your article is about ready to publish. So we need to get you access to the site so you can post in draft. I sent you an invite to the site. You need to accept it and you are off to the races :-) Pre-publishing checklist: (updated 1/29/2024)
Post-publishing checklist
|
Hi @bph @abhansnuk and all, thank you so much for all your help! I made the post available for public preview: https://developer.wordpress.org/news/?p=2702&preview=1&_ppp=a1d48c5d2b Social copy: Gain developer-level insights into the backend functions of user authentication in WordPress with a focus on extending and modifying each step: [Post link after publishing] |
The post is now published after I reviewed the post and made the following changes:
🙌 🎉Congrats to @Rajinsharwar for the first post published on the Developer Blog! |
schedule on social for 3/4 |
Discussed in #159
Originally posted by Rajinsharwar August 24, 2023
When developing user authentication systems for WordPress, it's really important to have a deep understanding of the login and registration processes. I am proposing a new article that will shed some light on the intricacies of these processes, and the underlying functions that ensure secure and seamless user experiences in WordPress sites.
I have also created a draft documentation for the purpose. Requesting your opinion and review on this:
https://docs.google.com/document/d/17N4uVctGa1pk8B2AcqY7KGGlY6yVV9azn85TsGJGALA/edit?usp=drivesdk
The text was updated successfully, but these errors were encountered: