Skip to content

Conversation

@dimaaan
Copy link

@dimaaan dimaaan commented Jan 26, 2026

Closes #268
Continues #269

@jsardev
Copy link
Owner

jsardev commented Jan 26, 2026

Hey @dimaaan, thank you for your contribution! Will take a look at the PR somewhere this week. I know it's small, but I didn't touch the project for a while already and need to catch up how this thing worked 😅

Copy link
Owner

@jsardev jsardev left a comment

Choose a reason for hiding this comment

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

LGTM

@jsardev
Copy link
Owner

jsardev commented Jan 27, 2026

@dimaaan Please rebase the PR with the current master - I had to fix CI to work again

@dimaaan
Copy link
Author

dimaaan commented Jan 27, 2026

@jsardev done

@jsardev jsardev changed the title support React 19 feat: support react@19 Jan 27, 2026
@jsardev
Copy link
Owner

jsardev commented Jan 27, 2026

@dimaaan Thank you! Unfortunately the tests do not pass due to old deprecated stack. I will need some time to update things to Vitest + RTL. Will let you know when it's ready so we could update the PR and make the tests pass.

@jsardev jsardev mentioned this pull request Jan 28, 2026
@jsardev
Copy link
Owner

jsardev commented Jan 28, 2026

@dimaaan Can you please merge master in, resolve some of the conflicts and create a changeset for this PR? 🙏

@jsardev
Copy link
Owner

jsardev commented Jan 29, 2026

@dimaaan Let me know if you have time to handle this. If not, I'll try to do this, just need info from you 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

reaptcha does not support react 19

2 participants