@kittycad/react-shared 1.1.0 → 1.1.1

package/README.md

@@ -54,6 +54,38 @@ npm run lint
 npm run fmt
 ```
 
+### Linking package locally
+
+Assumes your directory structure looks like
+```
+.
+├── modeling-app
+└── react-shared
+```
+
+```
+git clone git@github.com:KittyCAD/react-shared.git
+cd react-shared
+npm link
+npm install
+npm run build
+// edit your files
+// make sure to rebuild this after code changes!
+```
+
+```
+cd modeling-app
+rm -rf node_modules
+// Write "@kittycad/react-shared": "file:../react-shared", the package.json
+npm install
+npm install ../react-shared/ --install-links
+```
+
+#### Gotchas
+- using npm link can break with `vite` because vite will create `modeling-app/node_modules/.vite/deps` that are cached and will not be the correctly npm linked one. I needed to use the `file:` prefix to install the package to get around this `vite` issue.
+- `--install-links` commmand is required because the package has a peerDependenices list and React is in that list. We need to make sure that modeling-app and the package have the same react version otherwise you will run into problems.
+- Basic `npm link` and `npm link @kittycad/react-shared` will not work! `vite` bricks this!
+
 ### Available Components
 
 - **Button**: A customizable button component with multiple variants and sizes
@@ -93,4 +125,4 @@ This will trigger the GitHub Action to publish to npm.
 3. Make your changes
 4. Add tests and stories for new components
 5. Run `npm run lint` and `npm run type-check`
-6. Submit a pull request
+6. Submit a pull request

package/dist/index.d.ts

@@ -137,7 +137,7 @@ declare class BillingError {
 interface IBillingInfo {
     balance: number;
     allowance?: number;
-    userPaymentBalance: CustomerBalance;
+    userPaymentBalance?: CustomerBalance;
     paymentMethods: PaymentMethod[];
     isOrg: boolean;
     hasSubscription: boolean;

package/dist/index.esm.js

@@ -25826,11 +25826,20 @@ async function getBillingInfo(client) {
     }
     const subscriptions = await fetchBilling(j.get_user_subscription, { client });
     const org = await fetchBilling(L.get_user_org, { client });
-    const paymentMethods = await fetchBilling(j.list_payment_methods_for_user, { client });
-    if (BillingError.from(paymentMethods)) {
+    const hasOrgError = BillingError.from(org);
+    let paymentMethods = await fetchBilling(j.list_payment_methods_for_user, { client });
+    const isOrgUserWithoutPaymentMethod = !hasOrgError &&
+        BillingError.from(paymentMethods) &&
+        paymentMethods.error.type === EBillingError.NotOk &&
+        'status' in paymentMethods.error.response &&
+        paymentMethods.error.response.status === 404;
+    if (isOrgUserWithoutPaymentMethod) {
+        paymentMethods = [];
+    }
+    else if (BillingError.from(paymentMethods)) {
         return paymentMethods;
     }
-    if (!BillingError.from(org)) {
+    if (!hasOrgError) {
         // Org override here. TODO for api: remove the need for this
         // If the user is part of an org, they get infinite balance.
         return {