Set zero power snapshot balance when balance aggregator doesn't return any balance for an address

[aminlatifi]

impact-graph fills power snapshot balances by querying the balance aggregator as below

impact-graph/src/services/cronJobs/fillSnapshotBalances.ts

Lines 124 to 143 in bc3b435

	const balances =
	await getPowerBalanceAggregatorAdapter().getAddressesBalance({
	timestamp,
	addresses,
	});
	const groupByWalletAddress = _.groupBy(batch, item =>
	item.walletAddress.toLowerCase(),
	);
	const snapshotBalances = balances
	.map(balance =>
	groupByWalletAddress[balance.address.toLowerCase()].map(item => ({
	balance: balance.balance,
	powerSnapshotId: item.powerSnapshotId,
	userId: item!.userId,
	})),
	)
	.flat();
	await addOrUpdatePowerSnapshotBalances(snapshotBalances);

When the balance aggregator doesn't have any balance for an address, it doesn't return anything for that one. In the current implementation, the power balance snapshot for that user will remain empty forever. We must identify when no balance is returned for an address and fill it with zero!

[aminlatifi]

REf Giveth/giveth-dapps-v2#4326

[divine-comedian]

I think you're the tech lead for this one, assigning you @mohammadranjbarz

Also cc: @laurenluz for GIVeconomy Pm

[laurenluz]

@aminlatifi was this done already?

[aminlatifi]

@aminlatifi was this done already?

@laurenluz I am not informed if anything is done about it. @jainkrati Would be more informed.

[MoeNick]

@aminlatifi is it reviewed?

[aminlatifi]

@aminlatifi is it reviewed?

Yes, merged

[divine-comedian]

@aminlatifi is this issue done or does it require some QA or approval?

[aminlatifi]

@aminlatifi is this issue done or does it require some QA or approval?

It's merged to staging to address Giveth/giveth-dapps-v2#4326
On your call

[divine-comedian]

OK! @laurenluz I think you need to be the one to approve if this issue is Done, if not then what is required?