feat: v5 release blog post #1616
Conversation
✅ Deploy Preview for expressjscom-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Apparently you cannot reply to the review comment directly.
I did this intentionally. The audience for this specifically is broader than developers. It is a post for leaders and folks outside of our typical audience for a technical changelog. I was going to update the changelog itself to be the "punchier" thing. I will send this over to Robin and Jen to get their opinion on this before we make too many changes though, because if that is not hitting then I am totally open to taking a different approach. |
|
@wesleytodd I volunteer to translate your final post into my native language (Spanish) once it is published if you allow me and consider the translation necessary to generate a greater reach. |
|
I'm happy to see this started, even if just in draft version.... I suggest that the initial reviews focus primarily on content and technical accuracy. I'll be happy to work on stuff like voice, tone, wording, grammar, clarity, and so on once the basic content is solidified. This is just a matter of expediency and follows "levels of edit" best practice. To be clear, this is just my recommendation. Also, if @wesleytodd is not available to follow through on this for any reason, it could have multiple authors, or even an "Express TC" byline, if that makes sense. |
specifically, deleted the recommendations to use: res.send(body, status) res.json(obj, status) res.jsonp(obj, status)
Co-authored-by: Blake Embrey <[email protected]>
Co-authored-by: Blake Embrey <[email protected]>
Co-authored-by: Blake Embrey <[email protected]>
|
I made a full edit pass. I changed everything to third-person ("we") and changed the byline to "Wes Todd and Express TC" because IMO it's a group effort and also to emphasize that this release was indeed a group effort as well. If Wes really wants to keep just his name on the byline, that's fine, he can certainly change it back. There are still a three of "TO DO" items, and a mention of the changelog, which apparently is missing (as noted in the blog, lol). If we don't add the changelog before this is published, then we should remove "... for details, see the changelog." I resolved all the comments above that I addressed or commented accordingly. Hopefully it's clear. There are still a few unresolved comments that I couldn't address. We should try to do so before publishing if possible. |
UlisesGascon
force-pushed
the
v5-blog
branch
2 times, most recently
from
d4d3a79
to
f827d84
Compare
|
I made some final polish edits, including adding a links to the sections on each breaking change. LGTM! |
|
So this is finally ready. @expressjs/express-tc, @expressjs/triagers please review it 🙏. We plan to land this PR today (12h from now) unless there are request for changes. |
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
Co-authored-by: Jean Burellier <[email protected]>
|
btw... the date of the blog post is 2024-09-12. should we change it? |
|
@UlisesGascon yes, the date should be changed, maybe to tomorrow, October 16? |
|
I changed it to 10-15, which is today's date (in the US). The only way to do this, currently, is to rename the file. |
* feat: v5 release * delete duplicated removal notices specifically, deleted the recommendations to use: res.send(body, status) res.json(obj, status) res.jsonp(obj, status) * Update _posts/2024-09-12-v5-release.md Co-authored-by: Blake Embrey <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Blake Embrey <[email protected]> * typos, :name* change per blake Co-authored-by: Blake Embrey <[email protected]> * General edits, some review feedback * Add a missing TO DO * docs: removed ambiguous reference * docs: Removed TODO * docs: linting * docs: added sponsorship link * docs: added links to CVE-2024-45590, security triage team and last blog post * docs: improved metadata * docs: added explanation in the deprecated APIs * docs: added section for Migration and Security Guidance * Final polish, add toc links to breaking change sections * Change xref to migration guide from changelog * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Update _posts/2024-09-12-v5-release.md Co-authored-by: Jean Burellier <[email protected]> * Move file to change post date --------- Co-authored-by: Jon Church <[email protected]> Co-authored-by: Blake Embrey <[email protected]> Co-authored-by: Rand McKinney <[email protected]> Co-authored-by: Ulises Gascón <[email protected]> Co-authored-by: Jean Burellier <[email protected]>
Preview URL: https://deploy-preview-1616--expressjscom-preview.netlify.app/2024/09/12/v5-release