Remove propTypes definitions from Reviews By Product

[hritikchaudhary]

Fixes woocommerce/woocommerce#42362

Changes in the PR:

Removed propTypes definitions from Reviews By Product.

Testing

Automated Tests

• Changes in this PR are covered by Automated Tests.
  • Unit tests
  • E2E tests

User Facing Testing

• Do not include in the Testing Notes

WooCommerce Visibility

• WooCommerce Core
• Feature plugin
• Experimental

[hritikchaudhary]

Please check individual commit as even with git mv github is displaying block.tsx as new addition. All the relevant changes are in this commit - 51b76a9 rest commits are for renaming.
I renamed edit.tsx to block.tsx to keep the consistency with other blocks and it also helped resolve a TS error in index.tsx

[github-actions]

This PR has been marked as stale because it has not seen any activity within the past 7 days. Our team uses this tool to help surface pull requests that have slipped through review.

If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label.

[kmanijak]

Hi @hritikchaudhary!

Thanks a lot for your contribution!

By looking at the changes I don't think moving majority of the code from edit.tsx to block.tsx file is a good idea. There's a lot of code strictly related to Editor side (e.g. renderEditMode and whole Inspector Controls implementation).

I recommend to either stick only to removing propTypes definition or if you want to cleanup the code, split the block definition and Editor related code into block.tsx and edit.tsx. 🙏

---

Also, I’d like you to know that this weekend (8-9 December) WooCommerce Blocks repo will be migrated to WooCommerce monorepo.

However, open PRs won’t be migrated. That means open PRs have to be merged by the end of this week or otherwise, you’ll have to open/recreate a new PR in a monorepo post-migration if you’re interested in finalizing this contribution.

Thanks!

[github-actions]

This PR has been marked as stale because it has not seen any activity within the past 7 days. Our team uses this tool to help surface pull requests that have slipped through review.

If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label.