search

bitcoinvsalts / react-native-firebase-starter

The ultimate React Native starter using Firebase, Mobx, CodePush, OneSignal made with ♥
http://jsapp.me
MIT License
287 stars 56 forks source link

Posts are owned by User Name rather then Users.uuid #8

Closed bartonhammond closed 7 years ago

bartonhammond commented 7 years ago

Test: register with same name "John Doe" with two different email accounts. The posts are stored w/ user name rather UUID.

Note: If there is a User Name update screen provided, and the name was changed, then the Posts would be orphaned. Better to store the posts using the UUID of the User

bitcoinvsalts commented 7 years ago

I did not find the best solution concerning this issue yet. I tried to flat joined /posts and /users but it slows down the feed loading by a lot. Reading the username from /users for each post is very time and connection consuming. I am open for any suggestion concerning this.

bitcoinvsalts commented 7 years ago

I am trying to keep the feed load as simple as possible for now. I keep this note in mind for future development but thank you for the suggestion.

Hervé