Add try/catch logic so UI resets "saving" flags when a network call fails. (#2)

Previously, if a network call failed, the system would think the UI was still in a "saving" state. Now it turns off the "saving" flag even when the call fails.

Author: AndrewSouthpaw

CHANGELOG.md

@@ -0,0 +1,15 @@
+# Changelog
+
+## Unreleased
+
+## [0.0.5](https://github.com/AndrewSouthpaw/crud-muffins/releases/tag/v0.0.5) - 2020-12-03
+
+#### Added
+
+- Add try/catch logic so UI resets "saving" flags when network call fails. [#2](https://github.com/AndrewSouthpaw/crud-muffins/pull/2)
+
+## [0.0.1](https://github.com/AndrewSouthpaw/crud-muffins/releases/tag/v0.0.1) - 2020-08-10
+
+#### Added
+
+- Initial release

README.md

@@ -14,6 +14,16 @@ Because I got super tired of writing the basic state management of CRUD operatio
 1. Deleting an item
 1. ...and all the UI state associated with it (e.g. `isEditing`, `isSaving`, etc.)
 
+## Setup
+
+```sh
+$ yarn add crud-muffins
+```
+
+**This package is not transpiled and written with modern JS syntax**. You can add this package to the list of `node_modules` you transpile manually.
+
+Why? My current understanding is that it makes code bundles smaller overall when there's only *one* transpilation, as opposed to each module transpiling in its own context. Happy to be convinced otherwise!
+
 ## Usage
 
 The API will likely change quite a lot over the next several iterations. It's designed to work with any UI library, e.g. React, you just need to pass in the controls for reading/setting state.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "crud-muffins",
-  "version": "0.0.3",
+  "version": "0.0.5",
   "description": "A half-baked solution to UI CRUD operations.",
   "main": "dist/index.js",
   "scripts": {
@@ -24,7 +24,7 @@
   "devDependencies": {
     "@babel/cli": "^7.10.5",
     "@babel/core": "^7.10.5",
-    "@babel/preset-env": "^7.10.4",
+    "@babel/preset-env": "^7.12.1",
     "babel-plugin-ramda": "^2.0.0"
   },
   "dependencies": {

src/index.js

@@ -12,32 +12,36 @@ export const useCrudMuffinsUI = (ui, setUi) => {
   const setSavingNew = (value) => setSaving(value, '_new')
 
   const update = async (id, cb) => {
-    // TODO add try/catch
     setSaving(true, id)
 
-    const item = await cb()
-
-    setSaving(false, id)
-    setEditing(false, id)
-    return item
+    try {
+      const item = await cb()
+      setEditing(false, id)
+      return item
+    } finally {
+      setSaving(false, id)
+    }
   }
 
   const create = async (cb) => {
-    // TODO add try/catch
     setSavingNew(true)
-    const item = await cb()
-    setSavingNew(false)
-    return item
+    try {
+      const item = await cb()
+      return item
+    } finally {
+      setSavingNew(false)
+    }
   }
 
   const destroy = async (id, cb) => {
-    // TODO add try/catch
     setSaving(true, id)
 
-    await cb()
-
-    setSaving(false, id)
-    setEditing(false, id)
+    try {
+      await cb()
+      setEditing(false, id)
+    } finally {
+      setSaving(false, id)
+    }
   }
 
   return {