Skip to content

Conversation

@sansyrox
Copy link
Member

Summary

This PR is stacked over my previous PR and aims to solve the root issue from where the bug crept in.

I've created an abstraction file for the reason below:

This bug was induced as we are writing same code in 2 diff places(RNTesterApp.ios.js and RNTesterApp.android.js) violating the DRY principle.

Changelog

[General] [Changed] - Created an async abstraction

Test Plan

The app is running on both the emulators

@Xtremilicious
Copy link
Member

LGTM 🎉

@sansyrox sansyrox added the RNTester The issues associated with the RNTester project. label Aug 17, 2020
@anku255 anku255 merged commit 1c0e220 into mlh-fellowship-migrated-ui-changes Aug 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RNTester The issues associated with the RNTester project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants