fix(dashboard): use more idiomatic react-querybuilder code

[jakeboone02]

What changed? Why was the change needed?

The prepareRuleGroup function is not necessary to add ids to query objects generated from parseJsonLogic since v8.3.0 of react-querybuilder. This PR removes unnecessary prepareRuleGroup calls, updates the RQB dependency to v8.4.1, and moves an inline function prop to a stable reference to help with memoization.

I don't know how this project manages dependency updates, so maybe you can't merge this directly but if nothing else it can provide a couple of usage tips.

Screenshots

N/A

Expand for optional sections

Related enterprise PR

Special notes for your reviewer

I'm the maintainer of react-querybuilder and I just noticed this project. It's cool to see RQB being used so effectively! If you folks have any questions or feedback for me, please reach out!

EDIT: One more thing—the "here" link on this line is dead (gets a 404):

novu/CONTRIBUTING.md

Line 59 in da9336d

To learn more about the project structure and running the project locally, please have a look [here](https://docs.novu.co/community-support/introduction#run-novu-locally?utm_campaign=github-contrib).

[netlify]

✅ Deploy preview added

Name	Link
🔨 Latest commit	6f7bc60
🔍 Latest deploy log	https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/6845cbf1217716000852719b
😎 Deploy Preview	https://deploy-preview-8059.dashboard-v2.novu-staging.co
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[scopsy]

Thank you @jakeboone02 appreciate it a lot!