ecency / ecency-mobile

Ecency Mobile - reimagined social blogging, contribute and get rewarded (for Android and iOS)
https://ecency.com
MIT License
232 stars 69 forks source link

Alert.alert with non string argument causing app to crash. #2535

Closed noumantahir closed 1 year ago

noumantahir commented 1 year ago

Apparently there is some Alert.alert instance in app that is rarely causing app to crash in production, the issue is likely coming because a non string argument is passed to the alert.alert.

bugsnag ref: https://app.bugsnag.com/ecency/ecency-mobile/errors/61096df576a5b40007cffeea?filters[event.unhandled]=true&filters[event.since]=30d&filters[app.release_stage]=production

stackoverflow ref: https://stackoverflow.com/questions/52992606/value-for-title-can-not-be-cast-from-readablenativemap-to-string

feruzm commented 1 year ago

@noumantahir can we stringify everything, also we should cut string so it won't overflow screen with long and scary text for user.

noumantahir commented 1 year ago

@noumantahir can we stringify everything, also we should cut string so it won't overflow screen with long and scary text for user.

report was not specific, needed digging, but it was easy to find, already fixed the underlying source. There was just one possible instance that could lead to this crash