FIX - Error in Deploy

Author: VitorH19

.prettierrc

@@ -4,6 +4,5 @@
   "tabWidth": 2,
   "semi": false,
   "singleQuote": true,
-  "arrowParens": "always",
-  "editor.defaultFormatter": "esbenp.prettier-vscode"
+  "arrowParens": "always"
 }

server/routes/api/checkout.js

@@ -23,11 +23,11 @@ class CheckoutError extends Error {
 
 // Helper function to find or create a constituent
 async function findOrCreateConstituent(user) {
-  let constituent = await Constituent.query().where('email', user.email).first();
+  let constituent = await Constituent.query().where('email', user.email).first()
   if (!constituent) {
-    constituent = await Constituent.query().insert(user);
+    constituent = await Constituent.query().insert(user)
   }
-  return constituent;
+  return constituent
 }
 
 // Helper function to create a transaction
@@ -38,151 +38,134 @@ async function createTransaction(stripeTransactionId, constituentId, donation, s
     amount: donation,
     currency: 'usd',
     paymentMethod: 'credit_card',
-    status,
-  });
+    status
+  })
 }
 
 // Helper function to render letter HTML
 async function renderLetterHtml(letter, user) {
   letter.merge_variables = {
     ...letter.merge_variables,
     firstName: user.firstName,
-    lastName: user.lastName,
-  };
-  const template = await LetterTemplate.query().findById(letter.letter_template_id);
-  return Handlebars.render(letter.merge_variables, template.html);
+    lastName: user.lastName
+  }
+  const template = await LetterTemplate.query().findById(letter.letter_template_id)
+  return Handlebars.render(letter.merge_variables, template.html)
 }
 
 // Helper function to save letters for each delivery method
 async function saveLetters(transactionId, constituentId, letter, html, deliveryMethods) {
   for (const method of deliveryMethods) {
-    letter.trackingNumber = uuidv4(); // Generate unique tracking number
-    console.log(letter.trackingNumber);
+    letter.trackingNumber = uuidv4() // Generate unique tracking number
+    console.log(letter.trackingNumber)
 
     await Letter.query().insert({
       transactionId,
       constituentId,
       letterTemplate: html,
       deliveryMethod: method,
-      ...letter,
-    });
+      ...letter
+    })
   }
 }
 
 router.post('/create-transaction', async (req, res) => {
-  const stripe = new Stripe();
-  const db = require('../../db/connection');
+  const stripe = new Stripe()
+  const db = require('../../db/connection')
 
   try {
-    const { sessionId } = req.body;
+    const { sessionId } = req.body
 
     if (!sessionId) {
-      return res.status(400).json({ error: 'Session ID is required' });
+      return res.status(400).json({ error: 'Session ID is required' })
     }
 
-    const session = await stripe.checkout.sessions.retrieve(sessionId);
+    const session = await stripe.checkout.sessions.retrieve(sessionId)
 
     if (!session) {
-      return res.status(404).json({ error: 'Stripe session not found' });
+      return res.status(404).json({ error: 'Stripe session not found' })
     }
 
     const formattedTransaction = {
       stripe_transaction_id: sessionId,
       amount: session.amount_total,
       currency: session.currency,
       payment_method: session.payment_method_types[0] || 'unknown',
-      email: session.customer_details?.email || 'unknown',
-    };
+      email: session.customer_details?.email || 'unknown'
+    }
 
-    await db('transactions').insert(formattedTransaction);
+    await db('transactions').insert(formattedTransaction)
 
     return res.status(201).json({
       message: 'Transaction created successfully',
-      transaction: formattedTransaction,
-    });
+      transaction: formattedTransaction
+    })
   } catch (error) {
-    console.error('Error creating transaction:', error);
+    console.error('Error creating transaction:', error)
 
-    const statusCode = error instanceof CheckoutError ? 400 : 500;
+    const statusCode = error instanceof CheckoutError ? 400 : 500
 
-    return res.status(statusCode).json({ error: error.message });
+    return res.status(statusCode).json({ error: error.message })
   }
-});
-
-
+})
 
 // Refactored /create-checkout-session route
 router.post('/create-checkout-session', async (req, res) => {
-  const { donation, user, letter, deliveryMethods } = req.body;
-  const origin = req.get('origin');
+  const { donation, user, letter, deliveryMethods } = req.body
+  const origin = req.get('origin')
 
   try {
-    const presenter = new PaymentPresenter();
-    presenter.validatePaymentAmount(donation); // Validate donation amount
+    const presenter = new PaymentPresenter()
+    presenter.validatePaymentAmount(donation) // Validate donation amount
 
     // Handle free transactions
     if (donation === 0 && process.env.VUE_APP_EMPTY_TRANSACTIONS === 'on') {
-      const CHECKOUT_SESSION_ID = uuidv4();
-      const redirectUrl = `${origin}/complete?session_id=${CHECKOUT_SESSION_ID}`;
+      const CHECKOUT_SESSION_ID = uuidv4()
+      const redirectUrl = `${origin}/complete?session_id=${CHECKOUT_SESSION_ID}`
 
-      const constituent = await findOrCreateConstituent(user);
-      console.log(constituent.id);
+      const constituent = await findOrCreateConstituent(user)
+      console.log(constituent.id)
 
       const transaction = await createTransaction(
         `no-stripe-${uuidv4()}`,
         constituent.id,
         donation,
         'succeeded'
-      );
+      )
 
-      const html = await renderLetterHtml(letter, user);
-      await saveLetters(transaction.id, constituent.id, letter, html, deliveryMethods);
+      const html = await renderLetterHtml(letter, user)
+      await saveLetters(transaction.id, constituent.id, letter, html, deliveryMethods)
 
-      return res.status(200).json({ url: redirectUrl, sessionId: CHECKOUT_SESSION_ID }).end();
+      return res.status(200).json({ url: redirectUrl, sessionId: CHECKOUT_SESSION_ID }).end()
     }
 
     // Handle paid transactions
-    const redirectUrl = `${origin}/complete?session_id={CHECKOUT_SESSION_ID}`;
-    const cancelUrl = origin;
+    const redirectUrl = `${origin}/complete?session_id={CHECKOUT_SESSION_ID}`
+    const cancelUrl = origin
 
-    const stripe = new Stripe();
-    const session = await stripe.createCheckoutSession(donation, redirectUrl, cancelUrl);
+    const stripe = new Stripe()
+    const session = await stripe.createCheckoutSession(donation, redirectUrl, cancelUrl)
 
-    const constituent = await findOrCreateConstituent(user);
-    console.log(constituent.id);
+    const constituent = await findOrCreateConstituent(user)
+    console.log(constituent.id)
 
-    const transaction = await createTransaction(session.paymentIntent, constituent.id, donation);
+    const transaction = await createTransaction(session.paymentIntent, constituent.id, donation)
 
-    const html = await renderLetterHtml(letter, user);
-    await saveLetters(transaction.id, constituent.id, letter, html, deliveryMethods);
+    const html = await renderLetterHtml(letter, user)
+    await saveLetters(transaction.id, constituent.id, letter, html, deliveryMethods)
 
-    return res.status(200).json({ url: session.url, sessionId: session.id }).end();
+    return res.status(200).json({ url: session.url, sessionId: session.id }).end()
   } catch (error) {
-    console.error(error);
+    console.error(error)
 
-    const statusCode = error instanceof PaymentPresenterError ? 400 : 500;
-    return res.status(statusCode).json({ error: error.message }).end();
+    const statusCode = error instanceof PaymentPresenterError ? 400 : 500
+    return res.status(statusCode).json({ error: error.message }).end()
   }
-});
+})
 
 router.post('/process-transaction', async (req, res) => {
   try {
-    // const stripe = new Stripe()
-
-    // If livemode is false, disable signature checking
-    // and event reconstructionfor ease of testing.
     let event
-    /*
-    if (stripe.livemode) {
-      const signature = req.headers['stripe-signature']
-      if (!signature) throw new CheckoutError('No stripe signature on request!')
-
-      event = stripe.validateEvent(signature, req.rawBody)
-    } else {
-      event = req.body
-      // console.log(event)
-    }
-    */
     event = req.body
 
     if (!event) throw new CheckoutError('Unprocessable message')
@@ -193,8 +176,6 @@ router.post('/process-transaction', async (req, res) => {
 
     console.log(paymentIntent, amount, eventOutcome)
 
-    // We are not going to send letters from here just yet
-    // so we will record the transaction no matter the outcome.
     if (eventType !== 'payment_intent') {
       throw new CheckoutError(
         `Unexpected event! Received ${eventType} but it could not be processed.`
@@ -256,7 +237,6 @@ router.post('/process-transaction', async (req, res) => {
     }
 
     if (error instanceof StripeError) {
-      // Don't leak Stripe logging.
       console.error(error.message)
       error.message = 'Payment processing error'
     }
@@ -265,4 +245,4 @@ router.post('/process-transaction', async (req, res) => {
   }
 })
 
-module.exports = router
+module.exports = router