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 request for dashboard-v2-novu-staging pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	533122f