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

Conversation

@crazytonyli
Copy link
Contributor

@crazytonyli crazytonyli commented Jun 25, 2025

For context: p1747852183000959/1747165091.418189-slack-C011BKNU1V5

WordPressKit.zip

@crazytonyli crazytonyli requested a review from kean June 25, 2025 00:36
@dangermattic
Copy link
Collaborator

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

Generated by 🚫 Danger

Copy link
Contributor

@kean kean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Is this a new case? Why wasn't it needed before?

@crazytonyli
Copy link
Contributor Author

Nope, the error code has been there for at least 8 years. I suppose no one considered adding error handling code for it.

@crazytonyli crazytonyli merged commit 93f0994 into trunk Jun 26, 2025
5 of 7 checks passed
@crazytonyli crazytonyli deleted the add-error-code-reauthentication-required branch June 26, 2025 01:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants