Skip to content

fix: build error for react native 77#203

Closed
Naturalclar wants to merge 1 commit intomatinzd:mainfrom
Naturalclar:fix/buildForRN77
Closed

fix: build error for react native 77#203
Naturalclar wants to merge 1 commit intomatinzd:mainfrom
Naturalclar:fix/buildForRN77

Conversation

@Naturalclar
Copy link

Fixes Build Error for android React Native version 0.77

@Naturalclar
Copy link
Author

hi @matinzd, could you take a look at this when you are available? thanks!


fun parseWriteRecords(reactRecords: ReadableArray): List<Record> {
val recordType = reactRecords.getMap(0).getString("recordType")
val recordType = reactRecords.getMap(0)!!.getString("recordType")
Copy link
Owner

Choose a reason for hiding this comment

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

Please use optional unwrapping instead of force unwrapping.

val list = mutableListOf<ReadableMap>()
for (i in 0 until size()) {
list.add(getMap(i))
list.add(getMap(i)!!)
Copy link
Owner

Choose a reason for hiding this comment

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

Same here

@souvikinator
Copy link

Hi, when can we get a release for this fix? it seems to persist.

@matinzd matinzd closed this May 19, 2025
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.

3 participants

Comments