[PR #778] [MERGED] ♻️ further autocomplete refactors #3322

Closed
opened 2026-02-28 20:39:53 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/778
Author: @MatissJanis
Created: 3/18/2023
Status: Merged
Merged: 3/18/2023
Merged by: @MatissJanis

Base: masterHead: matiss/autocomplete-2


📝 Commits (5)

  • 6a6ac4e ♻️ further autocomplete refactors
  • db7d60b Cleaning up the diff
  • 56149d2 Fix lint
  • 7643d16 Shift control of inputValue back to PayeeAutocomplete
  • b4f7c2c Merge branch 'master' into matiss/autocomplete-2

📊 Changes

8 files changed (+238 additions, -99 deletions)

View changed files

📝 packages/desktop-client/src/components/accounts/TransactionsTable.js (+7 -1)
📝 packages/desktop-client/src/components/schedules/EditSchedule.js (+5 -1)
📝 packages/desktop-client/src/components/util/GenericInput.js (+21 -5)
packages/loot-design/src/components/NewAccountAutocomplete.js (+62 -0)
packages/loot-design/src/components/NewAutocomplete.js (+107 -0)
📝 packages/loot-design/src/components/NewPayeeAutocomplete.js (+24 -91)
📝 packages/loot-design/src/components/modals/EditField.js (+6 -1)
upcoming-release-notes/778.md (+6 -0)

📄 Description

Further iterations on the new autocomplete.

  1. Created AccountAutocomplete
  2. Started using new autocomplete in GenericInput (used for notes field)
  3. Extracted common functionality between the three new autocompletes to a generic component: Autocomplete

🔄 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/778 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 3/18/2023 **Status:** ✅ Merged **Merged:** 3/18/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/autocomplete-2` --- ### 📝 Commits (5) - [`6a6ac4e`](https://github.com/actualbudget/actual/commit/6a6ac4ea40ecec1dd771d2fd40ceefbe603b8980) :recycle: further autocomplete refactors - [`db7d60b`](https://github.com/actualbudget/actual/commit/db7d60bb6cb8362b278db1ab813cdd9a937623b5) Cleaning up the diff - [`56149d2`](https://github.com/actualbudget/actual/commit/56149d265eaa5707d43af3a4b2a701e8fa533a44) Fix lint - [`7643d16`](https://github.com/actualbudget/actual/commit/7643d16bea25deb6bc995e753a24c1b81dc188fd) Shift control of inputValue back to PayeeAutocomplete - [`b4f7c2c`](https://github.com/actualbudget/actual/commit/b4f7c2cb9ed06908b822e69a51495d169ef243e7) Merge branch 'master' into matiss/autocomplete-2 ### 📊 Changes **8 files changed** (+238 additions, -99 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/accounts/TransactionsTable.js` (+7 -1) 📝 `packages/desktop-client/src/components/schedules/EditSchedule.js` (+5 -1) 📝 `packages/desktop-client/src/components/util/GenericInput.js` (+21 -5) ➕ `packages/loot-design/src/components/NewAccountAutocomplete.js` (+62 -0) ➕ `packages/loot-design/src/components/NewAutocomplete.js` (+107 -0) 📝 `packages/loot-design/src/components/NewPayeeAutocomplete.js` (+24 -91) 📝 `packages/loot-design/src/components/modals/EditField.js` (+6 -1) ➕ `upcoming-release-notes/778.md` (+6 -0) </details> ### 📄 Description Further iterations on the new autocomplete. 1. Created `AccountAutocomplete` 2. Started using new autocomplete in `GenericInput` (used for notes field) 3. Extracted common functionality between the three new autocompletes to a generic component: `Autocomplete` --- <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-02-28 20:39:53 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#3322