chore(config): remove next.config.ts and update dependencies#65
Conversation
This commit deletes the next.config.ts file, which contained configuration for image remote patterns. Additionally, it adds the 'liquidjs' package to both package.json and package-lock.json, updating the project dependencies accordingly.
WalkthroughA new Changes
Poem
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
amplify/data/resource.ts (1)
171-186: Well-structured StoreTemplate model with some considerations.The model structure looks good and aligns with the liquidjs integration. However, consider these improvements:
Considerations for domain uniqueness:
- Multiple stores could potentially have the same domain. Consider if a uniqueness constraint on the domain field is needed.
Template validation suggestions:
- Consider adding validation for
templateKeyformat (e.g., enum or pattern matching)- The
templateDataJSON field offers flexibility but consider documenting expected schema or adding runtime validationThese are optional improvements that could enhance data integrity and developer experience.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (4)
amplify/data/resource.ts(1 hunks)app/(setup-layout)/first-steps/page.tsx(1 hunks)next.config.js(1 hunks)package.json(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Ejecutar pruebas unitarias
- GitHub Check: Aplicar Prettier
- GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (2)
next.config.js (1)
3-4: Good documentation improvement.The added comment clarifies the purpose of the images configuration section, improving code maintainability.
app/(setup-layout)/first-steps/page.tsx (1)
37-40: Well-aligned loading messages with new template engine functionality.The updated loading messages accurately reflect the new setup steps including template engine configuration, which aligns perfectly with the liquidjs dependency addition and new StoreTemplate model.
This commit deletes the next.config.ts file, which contained configuration for image remote patterns. Additionally, it adds the 'liquidjs' package to both package.json and package-lock.json, updating the project dependencies accordingly.
Summary by CodeRabbit
New Features
Enhancements
Chores
liquidjspackage to support template processing.