-
Notifications
You must be signed in to change notification settings - Fork 367
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
Release WP Job Manager 2.4.0 #2841
Conversation
Please perform the following tests with the built package in a new installation before publishing:
|
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.
Hey. This is looking good!
I left one question for the CSS change. :)
One thing I'm wondering here: Shouldn't this be 2.3.1? Which new features are we introducing here?
Also, shouldn't we update the class \WP_Job_Manager\Admin\Release_Notice
to reflect the news on wp-admin maybe?
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 discussed with @yscik on Slack and we found out that:
- It is better to be 2.4.0 on this case because we're removing IE11 support, so 2.3.1 will not highlight this
- The release notes on the notice doesn't need to be changed, because we're mostly fixing issues
Hence, I'm approving this PR. Let's release this!
✅ WP Job Manager 2.4.0 release created! |
1 similar comment
✅ WP Job Manager 2.4.0 release created! |
Important
Merging this PR will build and publish the new version automatically as a GitHub release, then deploy the new version on the WordPress.org plugin repository.
WP Job Manager 2.4.0
Note
These release notes between the two lines will be the final changelog entry for the release. Edit them freely here before merging.
Release Notes
Release
Note
Click 'Ready for Review', ping the team, review the PR and merge. Upon merging, automation will: