-
Notifications
You must be signed in to change notification settings - Fork 2k
Reader: Remove title from quick post. #108434
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
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
|
This PR modifies the release build for the following Calypso Apps: For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run |
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.
Curious, when I tested it, it was not working.
Approved, we just need to fix the related unit tests.
Fixes: READ-274
Proposed Changes
Remove title while publishing quick post.
Why are these changes being made?
Currently while publishing Quick Post we take first 57 characters of post content and use them as title. This is not good specially because of duplication.
One of the use case of editor is to make it similar to other social media platforms. Almost all of them don't require title so we are removing that as well.
Testing Instructions
Pre-merge Checklist