[PR #1212] [MERGED] Remove usage of Formik #65963

Closed
opened 2026-05-14 00:03:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1212
Author: @j-f1
Created: 6/27/2023
Status: Merged
Merged: 6/30/2023
Merged by: @j-f1

Base: masterHead: jed/no-formik


📝 Commits (8)

  • 968694d Fix LinkButton style
  • fae42dd Stop using Formik in CloseAccount modal
  • 4d542eb Stop using Formik in CreateLocalAccount
  • 668034b Clear errors on change/blur like Formik did
  • aed6bc7 Port to multiple state variables for errors
  • 8ea04c7 Remove Formik dependency
  • 8d0a3ea Add release note
  • a953681 Merge branch 'master' into jed/no-formik

📊 Changes

6 files changed (+247 additions, -293 deletions)

View changed files

📝 packages/desktop-client/package.json (+0 -1)
📝 packages/desktop-client/src/components/common.tsx (+3 -3)
📝 packages/desktop-client/src/components/modals/CloseAccount.js (+119 -131)
📝 packages/desktop-client/src/components/modals/CreateLocalAccount.js (+118 -118)
upcoming-release-notes/1212.md (+6 -0)
📝 yarn.lock (+1 -40)

📄 Description

This is a fairly large dependency, and we use it pretty lightly (just in the create/close account modals). It isn’t pulling its weight, so I’m removing it.


🔄 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/1212 **Author:** [@j-f1](https://github.com/j-f1) **Created:** 6/27/2023 **Status:** ✅ Merged **Merged:** 6/30/2023 **Merged by:** [@j-f1](https://github.com/j-f1) **Base:** `master` ← **Head:** `jed/no-formik` --- ### 📝 Commits (8) - [`968694d`](https://github.com/actualbudget/actual/commit/968694d8b876f35b7b487188570dbafb3f0362fe) Fix LinkButton style - [`fae42dd`](https://github.com/actualbudget/actual/commit/fae42dd67656fba31fe81ff52afb52d4895e648e) Stop using Formik in CloseAccount modal - [`4d542eb`](https://github.com/actualbudget/actual/commit/4d542eb71fef84e2a5a80e5bedb1a508552d0320) Stop using Formik in CreateLocalAccount - [`668034b`](https://github.com/actualbudget/actual/commit/668034b2ca62204aa629c3a28cd76db2cfcde72c) Clear errors on change/blur like Formik did - [`aed6bc7`](https://github.com/actualbudget/actual/commit/aed6bc7df37f55c93c3955b729bf82363788fd7b) Port to multiple state variables for errors - [`8ea04c7`](https://github.com/actualbudget/actual/commit/8ea04c79d400f5cbea48e08631307aabcba34378) Remove Formik dependency - [`8d0a3ea`](https://github.com/actualbudget/actual/commit/8d0a3eaefdbe2311a0bcd7de2ec1ac94dc4539e5) Add release note - [`a953681`](https://github.com/actualbudget/actual/commit/a9536810afa66f89d86da4a1da9759c5f6f1daff) Merge branch 'master' into jed/no-formik ### 📊 Changes **6 files changed** (+247 additions, -293 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/package.json` (+0 -1) 📝 `packages/desktop-client/src/components/common.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/modals/CloseAccount.js` (+119 -131) 📝 `packages/desktop-client/src/components/modals/CreateLocalAccount.js` (+118 -118) ➕ `upcoming-release-notes/1212.md` (+6 -0) 📝 `yarn.lock` (+1 -40) </details> ### 📄 Description This is a fairly large dependency, and we use it pretty lightly (just in the create/close account modals). It isn’t pulling its weight, so I’m removing it. --- <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-14 00:03:45 -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#65963