[PR #5685] [MERGED] [Maintenance] Remove usage of a raw variable in AccountAutocomplete component #55368

Closed
opened 2026-05-01 02:12:15 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5685
Author: @joel-jeremy
Created: 9/4/2025
Status: Merged
Merged: 9/5/2025
Merged by: @joel-jeremy

Base: masterHead: remove-variable-in-component-render-AccountAutocomplete


📝 Commits (6)

  • 29898bf Remove usage of a raw variable in AccountAutocomplete component
  • 981bd69 Remove usage of a raw variable in CategoryAutocomplete component
  • d1d14ca Release notes
  • e596d69 Fix typecheck error
  • b02b7b7 Fix typecheck errors
  • 21cf502 Remove CategoryAutocomplete changes

📊 Changes

2 files changed (+91 additions, -33 deletions)

View changed files

📝 packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx (+85 -33)
upcoming-release-notes/5685.md (+6 -0)

📄 Description

The AccountAutocomplete uses a lastItem variable during rendering which is hard to follow, this PR makes changes so that a variable that changes during render is no longer needed.


🔄 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/5685 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 9/4/2025 **Status:** ✅ Merged **Merged:** 9/5/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `remove-variable-in-component-render-AccountAutocomplete` --- ### 📝 Commits (6) - [`29898bf`](https://github.com/actualbudget/actual/commit/29898bfa9c8dabb8702dc7fb4f8180108e3f74d6) Remove usage of a raw variable in AccountAutocomplete component - [`981bd69`](https://github.com/actualbudget/actual/commit/981bd69997147d65b4d7e18330669d1230936878) Remove usage of a raw variable in CategoryAutocomplete component - [`d1d14ca`](https://github.com/actualbudget/actual/commit/d1d14caaad18a0ff307bee338437607706831adb) Release notes - [`e596d69`](https://github.com/actualbudget/actual/commit/e596d69508f5a5e27a036b52ed35e7f0cc07375c) Fix typecheck error - [`b02b7b7`](https://github.com/actualbudget/actual/commit/b02b7b737f4524b68b83bc9f140fe67c018f3997) Fix typecheck errors - [`21cf502`](https://github.com/actualbudget/actual/commit/21cf5020635ffb2f75a682f707eda31e8ff6fc37) Remove CategoryAutocomplete changes ### 📊 Changes **2 files changed** (+91 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx` (+85 -33) ➕ `upcoming-release-notes/5685.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> The AccountAutocomplete uses a `lastItem` variable during rendering which is hard to follow, this PR makes changes so that a variable that changes during render is no longer needed. --- <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-05-01 02:12:15 -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#55368