Skip to content

Conversation

@roxblnfk
Copy link
Collaborator

What was changed

Replaced react/promise v2 with forked and fixed ones.

Why?

We are stuck on React Promises v2, which the React team does not support at the level we need. We have issues with v2 code throwing deprecations on PHP 8.4 and PHP 8.5. We also have a need for the @yield annotation in promises.
I forked React Promise v2 and made the necessary changes without breaking backwards compatibility.

  • PHP 8.1+ compatibility
  • declare(strict_types=1); in all files
  • @yield annotations in the PromiseInterface
  • Enhanced type annotations

After that we can work on v3 and improve it to our requirements.

Checklist

  1. Part of react/promise v3 doesn't work fine #357

@roxblnfk roxblnfk requested a review from wolfy-j as a code owner August 26, 2025 19:38
@vercel
Copy link

vercel bot commented Aug 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
php Ready Ready Preview Comment Aug 27, 2025 6:42pm

@roxblnfk roxblnfk added this to the 2.16.0 milestone Aug 28, 2025
@roxblnfk roxblnfk mentioned this pull request Aug 28, 2025
@roxblnfk roxblnfk linked an issue Aug 28, 2025 that may be closed by this pull request
@roxblnfk roxblnfk removed this from the 2.16.0 milestone Aug 28, 2025
@roxblnfk roxblnfk marked this pull request as ready for review August 28, 2025 12:09
@roxblnfk roxblnfk merged commit a5f0489 into master Aug 28, 2025
218 of 226 checks passed
@roxblnfk roxblnfk deleted the replace-promises branch August 28, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

react/promise v3 doesn't work fine

3 participants