[Notifications] Add Like Post and Comment Inline Actions

[salimbraksa]

Fixes #22466

Description

This PR adds the Post Like inline action.

Challenges

Row Fade Animation

When toggling the post liked status, the notification row reloads with a fade animation. The reason behind this is because the notifications screen is using the WPTableViewHandler object, which observes Core Data changes. So when the notification object is updated locally, the WPTableViewHandler catches this update and reloads the associated row. I couldn't figure out an easy way to disable this behavior.

I have tried setting tableViewHandler.updateRowAnimation = .none but it didn't work.

Change Propagation

When a post's liked status is toggled anywhere on the app, this change isn't broadcasted to the Notifications screen. Here is how to reproduce this issue:

1. Navigate to Notifications screen
2. Tap on any new post notification
3. Expect the Reader Detail screen to appear
4. Like the post
5. Go back to Notifications screen
6. The post isn't liked

Ideally, when a post is liked on one screen, the same post should appear liked on all other loaded screens. I tested on Android and I was able to reproduce this issue.

Test Instructions

Posts

1. Run the Jetpack app and switch to the notifications tab.
2. Navigate to Notifications tab.
3. Expect to see the "Star" icon on new post notifications.
4. Tap the "Star" to like / un-like a new post notification.
5. Expect the like status to immediately change
6. Tap on the notification row.
7. Expect the Reader Detail screen to appear
8. Verify the post is liked

Comments

1. Run the Jetpack app and switch to the notifications tab.
2. Navigate to Notifications tab.
3. Expect to see the "Star" icon on comment notifications.
4. Tap the "Star" to like / un-like a comment.
5. Expect the like status to immediately change
6. Tap on the notification row.
7. Depending on whether the user can moderate comments:
   • User is admin: Expect the Comment Detail screen to appear.
   • User is not admin: Expect the Reader Comments screen to appear.
8. Verify the comment is liked.

Regression Notes

1. Potential unintended areas of impact Smoke test post like feature in other parts of the app ( e.g Reader )

2. What I did to test those areas of impact (or what existing automated tests I relied on) Manual testing

3. What automated tests I added (or what prevented me from doing so) N/A

PR submission checklist:

• [x] I have completed the Regression Notes.
• [x] I have considered adding unit tests for my changes.
• [x] I have considered adding accessibility improvements for my changes.
• [x] I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

[wpmobilebot]

📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.

	App Name	WordPress Alpha
	Configuration	Release-Alpha
	Build Number	pr22688-55200cd
	Version	24.3
	Bundle ID	org.wordpress.alpha
	Commit	55200cd4a79113ed1ec3b167d950dfa36666187e
	App Center Build	WPiOS - One-Offs #8981

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

[wpmobilebot]

📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.

	App Name	Jetpack Alpha
	Configuration	Release-Alpha
	Build Number	pr22688-55200cd
	Version	24.3
	Bundle ID	com.jetpack.alpha
	Commit	55200cd4a79113ed1ec3b167d950dfa36666187e
	App Center Build	jetpack-installable-builds #8011

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

[hassaanelgarem]

Liking a post crashes for me, I'm considering changing the approach so I will mark the PR as draft for now. cc @alpavanoglu

[dangermattic]

	1 Warning
:warning:	This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by :no_entry_sign: Danger

[alpavanoglu]

🐛 When the like status is changed from the details screen bottom bar, it doesn't reflect on the row after going back. This is still the case after a pull-to-refresh action.