-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat: add Version Support page and Commercial Support Options #1585
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
Thanks for the PR @ljharb, I made some changes on the description. BTW the CI failing error is not related to this PR. |
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.
LGTM
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.
I think we might want to nail down some of the LTS plans before we land this, but I am good with the changes here.
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.
@wesleytodd if we want to wait to land this, we should convert to draft
I will land it, seems like we are align with the changes. We can always open an additional PR to make more changes if needed :) |
Main Changes
Context
Related to expressjs/discussions#257
Details
Support page
Preview: https://deploy-preview-1585--expressjscom-preview.netlify.app/en/support/
Banner
Preview: https://deploy-preview-1585--expressjscom-preview.netlify.app/en/3x/api