Why both user/totalTweets and user/stats/stats/tweets.length ?

[stanbar]

Hi,
I wonder why do you use both user/totalTweets and user/stats/stats/tweets.length? What is the reason of keeping them in both places? And then when you display user's total tweets in user-header.tsx you add them together in

twitter-clone/src/components/user/user-header.tsx

Line 38 in c6bf6fb

(user?.totalTweets ?? 0) + (tweets?.length ?? 0),

?

I also wonder why are they managed separatelly, i.e. user.totalTweets is incremented/decremented on client-side only

twitter-clone/src/lib/firebase/utils.ts

Line 171 in c6bf6fb

export async function manageTotalTweets(

While user/stats/stats/tweets is incremented/decremented on both client-side

twitter-clone/src/lib/firebase/utils.ts

Line 210 in c6bf6fb

tweets: arrayUnion(tweetId),

and only decremented on server-side via Cloud functions onDelete trigger)

twitter-clone/functions/src/normalize-stats.ts

Line 5 in c6bf6fb

export const normalizeStats = regionalFunctions.firestore

[stanbar]

I figured out the answer. user/stats/stats/tweets doesn't keep track of tweets but of REtweets only. It's managed in separate doc users/{userId}/stats/stats as a optimisation, that way we don't have to fetch whole list of likes and retweets every time we want to get user doc.

I suggest renaming user/stats/stats/tweets to user/stats/stats/retweets as it is very confusing.