Codeinwp / Codeinwp/tweet-old-post
Possible changes to instant sharing custom messages feature
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 13
- Forks
- 15
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 8
Description
- Allow saving of custom instant messages for posts that are in Pending, so that when being reviewed, the post would already have custom messages.
Usecase:
Writer entering custom share messages
Admin altering those custom messages or choosing to publish as is
- Allow saving of custom instant messages for scheduled posts. Right now this feature solely depends on if the option is checked in the general settings of ROP. But if a post is a draft, and a user saves it as "scheduled" with custom instant share messages, then when that post changes from "scheduled" to "published" and the share is triggered. It should pull the custom share message one it exists.
Change this:
https://github.com/Codeinwp/tweet-old-post/blob/v8511/includes/admin/class-rop-admin.php#L707
Add check for custom messages here:
https://github.com/Codeinwp/tweet-old-post/blob/v8511/includes/admin/class-rop-admin.php#L786
Or here (might be better here):
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading includes/admin/class-rop-admin.php around lines 707 and 786, then inspect includes/admin/models/class-rop-queue-model.php around line 325. Trace how pending and scheduled posts retain custom instant share messages. Done means those messages remain available for review or publication and are used when a scheduled post is published and sharing is triggered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100