wordpress-mobile / wordpress-mobile/WordPress-iOS

To-Do Block: check/uncheck tasks from Reader doesn't work

Open
#16,792 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

[Pri] Low [Type] Task Reader
Dominant language
Swift
Stars
3.9k
Forks
1.2k
Avg merge
23h 51m
Merged PRs (30d)
58

Description

Android issue: https://github.com/wordpress-mobile/WordPress-Android/issues/14976

Expected behavior

When I tap on a task, the task is marked as done (strikethrough) and my @ is added after it between parenthesis. Just like it is on the web.

Actual behavior

Tapping on the task does nothing.

Steps to reproduce the behavior
  1. Create a post in an internal P2 and add a To-Do list block
  2. Add a few tasks
  3. Open this post on Reader
  4. Tap on any of the tasks
  5. Nothing happens
Technical details

In order for this to work on mobile Reader, we would need:

  1. When a task is tapped, the native application needs to trigger a request
  2. We need to manipulate the HTML to mark the task as "done"

The big issue I see here is the request. By doing a quick investigation (you can just use your browser to do the same) it seems to me that what happens when a task is marked as done is that a request to update the content of this post is triggered. This is tricky to do from the Reader because sometimes what we have is just the HTML output (even with some stuff cleaned). If we use that to update the post, we might end up breaking it (and removing the block declarations).

Ps.: it worths noting that this issue doesn't affect end users, given to-do block is used on internal P2s.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the issue in Reader using the listed P2 post and compare it with the linked WordPress-Android issue. Investigate the browser request made when a task is marked done and how Reader obtains the post HTML. Done means tapping a task marks it complete and adds the user's @ without removing block declarations or breaking the post.

Written by the indexing model from the issue text.

Assessment

Tech stack
swift
Domain
mobile-dev
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.