From 4f0091ecc859640a329100c348b9a1c7af849f0b Mon Sep 17 00:00:00 2001
From: Matt Stover
Date: Wed, 26 Jun 2024 16:02:50 -0700
Subject: [PATCH 1/3] fix: prevent massive error logs due to failed receipt
query when no transactionId is present
---
src/pages/Thanks/ThanksPage.vue | 38 +++++++++++++++++++++++----------
1 file changed, 27 insertions(+), 11 deletions(-)
diff --git a/src/pages/Thanks/ThanksPage.vue b/src/pages/Thanks/ThanksPage.vue
index c1e2ec0eb3..47c46f9c78 100644
--- a/src/pages/Thanks/ThanksPage.vue
+++ b/src/pages/Thanks/ThanksPage.vue
@@ -45,17 +45,6 @@
We've emailed your order confirmation to you.
-
-
-
- Please log in to see your receipt.
-
-
- Log in to continue
-
-
+
+
+
+ Please log in to see your receipt.
+
+
+ Log in to continue
+
+
+
Date: Wed, 26 Jun 2024 16:03:08 -0700
Subject: [PATCH 2/3] chore: update local dev with caddy docs
---
README.md | 43 +++++++++++++++++++++++++++++++++++++++++--
1 file changed, 41 insertions(+), 2 deletions(-)
diff --git a/README.md b/README.md
index 2649d6896a..0ff0c61267 100644
--- a/README.md
+++ b/README.md
@@ -12,13 +12,52 @@ The Kiva UI project is bound by a [Code of Conduct](code_of_conduct.md).
Kiva welcomes outside contributions to our UI repository. If you have any ideas for a feature or improvement, create an issue and we can discuss whether it makes sense to create a pull request. Thanks for the help!
-## Build Setup for localhost (outside of a Kiva VM or Server)
+# Local Development Setup with Caddy
+
+> IMPORTANT NOTE: Turn off Docker if it's running! We have a perpetually running network related to Tilt that will prevent Caddy from starting. I did consider using Tilt and docker-compose but it would require rewriting alot of the Monolith TiltVM setup so this just bypasses it all.
+
+### Required Dependencies
+
+1. Add `127.0.0.1 kiva-ui.local` to your `/etc/hosts` file on your mac
+ - Auth0 configs are already in place to support this domain in the dev tenant
+
+2. Install Caddy
+
+ `brew install caddy`
+
+3. Start Caddy from the root of the ui repo
+
+ `caddy start` to run in the background or `caddy run` keep the terminal live for additional monitoring
+
+4. In a separate terminal at the root of the ui repo
+``` bash
+# Set you node version using nvm
+$ nvm use
+
+# install dependencies
+$ npm ci
+
+# install husky git hooks (NOTE: This step only needs to be done once on first setup and powers pre-commit linting)
+$ npx husky install
+
+$ npm run dev -- --config=dev-custom-host
+
+# The local dev URL is now: https://kiva-ui.local/, but make sure to access a page actually run by UI, for example https://kiva-ui.local/lend-by-category/women
+
+```
+
+5. To stop Caddy when you're done
+
+ `caddy stop`
+
+
+## Build Setup for localhost develoment (outside of a Tilt or Server environments)
``` bash
# DEV MODE
# install dependencies
-$ npm install
+$ npm ci
# install husky git hooks (powers pre-commit linting)
$ npx husky install
From e344ec75b6f2359c860d879c64f1b89c3076e1a0 Mon Sep 17 00:00:00 2001
From: Matt Stover
Date: Wed, 26 Jun 2024 17:47:32 -0700
Subject: [PATCH 3/3] fix: log warning when prefetch or readQuery are skipped
due to missing transaction id
---
src/pages/Thanks/ThanksPage.vue | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/src/pages/Thanks/ThanksPage.vue b/src/pages/Thanks/ThanksPage.vue
index 47c46f9c78..875b1053f9 100644
--- a/src/pages/Thanks/ThanksPage.vue
+++ b/src/pages/Thanks/ThanksPage.vue
@@ -255,6 +255,10 @@ export default {
// Check if transactionId is null, resolve the promise if missing
if (!transactionId) {
+ logFormatter(
+ 'Thanks page preFetch skipped due to missing transaction_id.',
+ 'warning',
+ );
return Promise.resolve();
}
@@ -400,6 +404,10 @@ export default {
// Check if transactionId is null, exit if missing
if (!transactionId) {
+ logFormatter(
+ 'Thanks page readQuery skipped due to missing transaction_id.',
+ 'warning',
+ );
return false;
}