P RAHUL

Results 9 comments of P RAHUL

Hey @Ainali, I’ve made the requested updates: -->Split the office (P2388, P1313) and person-based properties (P488, P169, P1037, P3975) into separate blocks. -->Removed the COALESCE fallback, since the frontend handles...

Hi @Ainali — I’ve now removed the COALESCE fallback from ?leadBy as requested. Everything else is unchanged. Please review when you get a chance. Thanks!

Hi @Ainali, I’ve pushed the latest commit removing the COALESCE fallback as you requested. All checks are passing and there are no conflicts. Could you please review and approve when...

Thanks @Ainali for the approval and local test confirmation. I’ll wait for @Abbe98’s review to make sure everything is aligned before merge. Appreciate your time!

Thanks, @Ainali! I ran local tests on my end too — checked multiple country entries to confirm the fallback logic and lead_by additions behave as expected without breaking anything else....

Thanks everyone for reviewing. Everything looks good on my end and the tests confirm expected behavior. I’ll hold off until @Abbe98 gives the final go-ahead. Really appreciate the quick feedback...

Hi @Abbe98 --> just following up to see if you had a chance to review this. It’s already approved by @Ainali and verified locally on both ends. Appreciate your time.

Thanks @Abbe98 I will cleaned up the extra line breaks as requested. Please let me know if it looks good now.

Hi @Abbe98 — just following up to check if the cleanup looks good now. Everything’s updated and verified locally. Appreciate your time and review!