[PR #2029] [MERGED] ♻️ (eslint) enable 'react/no-children-prop' rule and fix issues #38506

Closed
opened 2026-04-23 11:34:09 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2029
Author: @MatissJanis
Created: 12/4/2023
Status: Merged
Merged: 12/5/2023
Merged by: @MatissJanis

Base: masterHead: matiss/eslint-imp


📝 Commits (2)

  • 0dd3a3d ♻️ (eslint) enable 'react/no-children-pro' rule and fix issues
  • fce0771 Release notes

📊 Changes

12 files changed (+36 additions, -27 deletions)

View changed files

📝 .eslintrc.js (+0 -1)
📝 packages/desktop-client/src/components/NotesButton.tsx (+3 -5)
📝 packages/desktop-client/src/components/ScrollProvider.tsx (+3 -4)
📝 packages/desktop-client/src/components/Titlebar.tsx (+3 -4)
📝 packages/desktop-client/src/components/budget/report/ReportContext.tsx (+3 -2)
📝 packages/desktop-client/src/components/budget/rollover/RolloverContext.tsx (+3 -2)
📝 packages/desktop-client/src/hooks/useSelected.tsx (+3 -5)
📝 packages/loot-core/src/client/data-hooks/accounts.tsx (+3 -1)
📝 packages/loot-core/src/client/data-hooks/payees.tsx (+3 -1)
📝 packages/loot-core/src/client/data-hooks/schedules.tsx (+5 -1)
📝 packages/loot-core/src/client/query-hooks.tsx (+1 -1)
upcoming-release-notes/2029.md (+6 -0)

📄 Description

Enabling react/no-children-prop rule and fixing the issues.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/2029 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 12/4/2023 **Status:** ✅ Merged **Merged:** 12/5/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/eslint-imp` --- ### 📝 Commits (2) - [`0dd3a3d`](https://github.com/actualbudget/actual/commit/0dd3a3dbfdd22d90fde669a2b2b833cd8b2d431e) :recycle: (eslint) enable 'react/no-children-pro' rule and fix issues - [`fce0771`](https://github.com/actualbudget/actual/commit/fce0771f58f34daf6b66f6154b67c902b926127f) Release notes ### 📊 Changes **12 files changed** (+36 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+0 -1) 📝 `packages/desktop-client/src/components/NotesButton.tsx` (+3 -5) 📝 `packages/desktop-client/src/components/ScrollProvider.tsx` (+3 -4) 📝 `packages/desktop-client/src/components/Titlebar.tsx` (+3 -4) 📝 `packages/desktop-client/src/components/budget/report/ReportContext.tsx` (+3 -2) 📝 `packages/desktop-client/src/components/budget/rollover/RolloverContext.tsx` (+3 -2) 📝 `packages/desktop-client/src/hooks/useSelected.tsx` (+3 -5) 📝 `packages/loot-core/src/client/data-hooks/accounts.tsx` (+3 -1) 📝 `packages/loot-core/src/client/data-hooks/payees.tsx` (+3 -1) 📝 `packages/loot-core/src/client/data-hooks/schedules.tsx` (+5 -1) 📝 `packages/loot-core/src/client/query-hooks.tsx` (+1 -1) ➕ `upcoming-release-notes/2029.md` (+6 -0) </details> ### 📄 Description Enabling `react/no-children-prop` rule and fixing the issues. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-23 11:34:09 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#38506