Skip to content

feat: relocate data logic to modules folder#30

Merged
rylanharper merged 4 commits intomasterfrom
feat/relocate-data-logic
Sep 19, 2025
Merged

feat: relocate data logic to modules folder#30
rylanharper merged 4 commits intomasterfrom
feat/relocate-data-logic

Conversation

@rylanharper
Copy link
Owner

Description

This PR relocates the Shopify and Klaviyo data logic into their respective /modules folders. Certain Nuxt composables, like $fetch and creatError, did properly reach inside the /data folder, since its not an official folder recognized by Nuxt. This causes type readability in VS code, which is sort of annoying.

I have added a new resources folder to each module that contains all of its corresponding data logic.

Type of change

  • Feature
  • Fix
  • Refactor
  • Documentation

Summary

  • Move Shopify and Klaviyo data to into their respective /modules folders, under /resources
  • Update GraphQL Client, includes new cachable param
  • Update Codegen documents path
  • Update documentation
  • Update types

@vercel
Copy link

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
nitrogen Ready Ready Preview Comment Sep 19, 2025 8:33pm

@netlify
Copy link

netlify bot commented Sep 19, 2025

Deploy Preview for nitrooogen ready!

Name Link
🔨 Latest commit 809881a
🔍 Latest deploy log https://app.netlify.com/projects/nitrooogen/deploys/68cdbe0d56134c0008d9a216
😎 Deploy Preview https://deploy-preview-30--nitrooogen.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rylanharper rylanharper merged commit 7dce7df into master Sep 19, 2025
6 of 7 checks passed
@rylanharper rylanharper deleted the feat/relocate-data-logic branch September 19, 2025 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant