Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Update Reader APIs #820

Merged
merged 2 commits into from
Dec 19, 2024
Merged

Update Reader APIs #820

merged 2 commits into from
Dec 19, 2024

Conversation

@kean
Copy link
Contributor

@kean kean commented Oct 18, 2024 •
edited
Loading

Description

  • Add support for changing the Discover streams (required for first-posts)
  • Add support for site icons in Discover

Testing Details

PR: wordpress-mobile/WordPress-iOS#23676

  • Check that site icons are loading in Discover
  • Check that "First Posts" stream works and show the same posts as on the web
  • Please check here if your pull request includes additional test coverage.
  • I have considered if this change warrants release notes and have added them to the appropriate section in the CHANGELOG.md if necessary.

@dangermattic
Copy link
Collaborator

dangermattic commented Oct 18, 2024

1 Warning
⚠️ Package.swift was changed without updating its corresponding Package.resolved. Please resolve the Swift packages in Xcode.

Generated by 🚫 Danger

@kean kean requested a review from crazytonyli October 18, 2024 18:07
@kean kean merged commit 37b3a3b into wpios-edition Dec 19, 2024
1 check passed
@kean kean deleted the task/reader-discover branch December 19, 2024 20:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@crazytonyli crazytonyli crazytonyli approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kean @dangermattic @crazytonyli