wordpress-mobile / wordpress-mobile/WordPress-Android
Reader: Cleanup copy missing fields logic for a post
Nobody has claimed this yet.
- Dominant language
- Kotlin
- Stars
- 3.2k
- Forks
- 1.4k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 69
Description
While fixing the duplicate posts issue here, it was noticed that:
-
PR #3165 copied fields from the local post to the post from server to have missing fields like
word_countin theread/sites/$siteid/posts/$postidendpoint so the existing information isn't lost. -
PR #4472 removed the
word_countfield.
However, the copy fields logic still remained in the existing code.
The goal of this issue is to investigate any other missing fields that we need and is not returned by the endpoint, ideally get them from the endpoint and get rid of this hack.
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 in WordPress/src/main/java/org/wordpress/android/ui/reader/actions/ReaderPostActions.java at the existing copy-fields logic, then compare it with the read/sites/$siteid/posts/$postid endpoint and the fields returned there. Identify any fields still genuinely missing, ensure they are supplied by the endpoint where needed, and remove the obsolete client-side workaround.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, java
- Domain
- api, mobile
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100