Skip to content

Fix build on RN 0.81#206

Merged
mrousavy merged 1 commit into
mrousavy:masterfrom
riteshshukla04:master
Aug 13, 2025
Merged

Fix build on RN 0.81#206
mrousavy merged 1 commit into
mrousavy:masterfrom
riteshshukla04:master

Conversation

@riteshshukla04

@riteshshukla04 riteshshukla04 commented Aug 12, 2025

Copy link
Copy Markdown
Contributor
image

Build fails with above error . This should fix it

@mrousavy mrousavy merged commit b37ddb8 into mrousavy:master Aug 13, 2025
1 of 4 checks passed
@mrousavy

Copy link
Copy Markdown
Owner

dope thanks!

@elyobo

elyobo commented Sep 18, 2025

Copy link
Copy Markdown

Ha, just came to open a PR for this as well - any chance of a release? I'm using patch-package to apply the change locally but definitely preferable not to.

@tsatsujnr139

tsatsujnr139 commented Sep 20, 2025

Copy link
Copy Markdown

This didn't solve the issue for me, which makes sense because this is an override. I had to append .toMutableMap() to the builder instead.

 override fun getExportedCustomDirectEventTypeConstants(): MutableMap<String, Any>? {
    return MapBuilder.builder<String, Any>()
        .put(LoadErrorEvent.EVENT_NAME, MapBuilder.of("registrationName", "onLoadError"))
        .put(LoadStartEvent.EVENT_NAME, MapBuilder.of("registrationName", "onLoadStart"))
        .put(LoadEndEvent.EVENT_NAME, MapBuilder.of("registrationName", "onLoadEnd"))
        .build()
        .toMutableMap()
  }

@mrousavy

Copy link
Copy Markdown
Owner

@tsatsujnr139 reading this just now - are you sure this doesn't fix the issue?

If yes, can you please create a PR with your change instead to make it work?

Thanks

@tsatsujnr139

Copy link
Copy Markdown

Yes quite sure it didn't solve it...

I'll gladly create the PR after I test again to be sure

@mrousavy

Copy link
Copy Markdown
Owner

I'll gladly create the PR after I test again to be sure

Thank you!

@riteshshukla04

riteshshukla04 commented Dec 11, 2025

Copy link
Copy Markdown
Contributor Author

I am not sure the entire convo. But This PR fixed the issue

https://github.com/Jellify-Music/App/blob/main/patches/react-native-blurhash%2B2.1.1.patch

We are using the same patch in the app for couple of months now and it is working flawlessly

@elyobo

elyobo commented Dec 11, 2025

Copy link
Copy Markdown

Yeah, the fix is already good for us.

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.

4 participants