Skip to content

Docker file #31

Merged
VisibleNasir merged 3 commits into
mainfrom
MerchantQRCode
Nov 9, 2025
Merged

Docker file #31
VisibleNasir merged 3 commits into
mainfrom
MerchantQRCode

Conversation

@VisibleNasir

Copy link
Copy Markdown
Collaborator

No description provided.

- Created migration scripts to drop existing `recharge_orders` and `recharge_plans` tables and create new `RechargeOrder` and `RechargePlan` tables with updated schemas.
- Introduced new reward and referral models with migrations for `Reward` and `Referral` tables, including enums for reward types and statuses.
- Added migration for refund requests with new `WrongSendRequest` table and updated `p2pTransfer` schema.
- Updated Prisma schema to include new models and relationships for recharge, rewards, and referral systems.
- Seeded initial data for rewards, including scratch cards, cashback, referral rewards, and milestone achievements.
- Enhanced UI components for better styling and accessibility, including updates to `TextInput`, `AnimatedTestimonials`, `Avatar`, `Button`, and `SidebarLink`.
….json

- Added eslint-config-turbo and eslint-plugin-turbo to the root package.json
- Updated prisma version from 6.18.0 to 6.19.0 in packages/db/package.json
@VisibleNasir VisibleNasir requested a review from Copilot November 9, 2025 09:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request implements a comprehensive theme and UI overhaul for the CalxSecure payment platform, introducing dark mode support, new UI components, and infrastructure improvements.

Key Changes

  • Added dark mode support with theme toggling across user and merchant apps
  • Introduced new UI components (GlareCard, ShootingStars, StarsBackground, HoverBorderGradient, BackgroundBeamsWithCollision)
  • Updated Docker configuration for better setup and database initialization
  • Added Firebase SDK (v12.5.0) and upgraded dependencies (Prisma 6.18.0 → 6.19.0, eslint-config-turbo/eslint-plugin-turbo 1.13.4 → 2.6.0)
  • Replaced HTML anchor tags with Next.js Link components for better routing
  • Modified styling across all components to support light/dark themes

Reviewed Changes

Copilot reviewed 46 out of 48 changed files in this pull request and generated 12 comments.

Show a summary per file
File Description
packages/ui/src/sidebar.tsx Replaced <a> with <Link>, removed color classes for theme support
packages/ui/src/button.tsx Updated background colors to support dark mode
packages/ui/src/avatar.tsx Added border and dark mode background styling
packages/ui/src/TextInput.tsx Removed hardcoded background/text colors for theme compatibility
packages/db/package.json Upgraded Prisma from 6.18.0 to 6.19.0
package.json Added firebase 12.5.0, eslint-config-turbo/plugin 2.6.0, @types/ioredis
docker-compose.yml Complete rewrite: healthchecks, setup service, Prisma migration automation
apps/user-app/lib/redis.ts Added type safety with IORedisClient type alias and explicit casts
apps/user-app/lib/firebase.ts Removed comment-only line
apps/user-app/components/* Extensive styling updates for dark mode, replaced inline styles with theme-aware classes
apps/user-app/app/home/page.tsx Removed unused image imports, updated nav items, added new background components
apps/user-app/app/auth/* Updated styling for dark mode, improved form layouts
apps/user-app/app/(dashboard)/* Added dark mode support, improved layouts, added loading state
apps/merchant-app/components/* Added new UI components (ShootingStars, StarsBackground, BackgroundBeamsWithCollision)
apps/merchant-app/app/home/page.tsx Major redesign with new background effects and theme support
Dockerfile Simplified: switched to Node 20, added PostgreSQL client
Comments suppressed due to low confidence (1)

apps/merchant-app/app/(dashboard)/bills/page.tsx:68

  • Corrected spacing in 'cursor-pointer' (currently has space: 'cursor -pointer').

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 11 to +17
redis = new Redis(process.env.REDIS_URL || "redis://localhost:6379", {
lazyConnect: true,
maxRetriesPerRequest: 1,
retryStrategy: () => null, // Don't retry
enableOfflineQueue: false,
enableReadyCheck: false,
});
}) as IORedisClient;

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cast to IORedisClient on line 17 is unnecessary. The new Redis() constructor already returns an instance of type Redis, which is the same as InstanceType<typeof Redis>. Remove the explicit cast for cleaner code.

Copilot uses AI. Check for mistakes.
className="w-full bg-zinc-400 hover:bg-zinc-500 disabled:bg-zinc-600/50 disabled:cursor-not-allowed"
>
Pay INR{amount || 0}
Pay

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The button text 'Pay' is incomplete. It previously showed 'Pay INR{amount || 0}' which provided important context to the user about the amount being paid. This change removes critical payment information from the UI.

Suggested change
Pay
Pay INR{amount || 0}

Copilot uses AI. Check for mistakes.
<div
className={`p-4 rounded-lg text-sm font-medium ${
message.includes("") ? "bg-green-100 text-green-700" : "bg-red-100 text-red-700"
message.includes("") ? "bg-green-100 text-green-700" : "bg-red-100 text-red-700"

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The condition message.includes(\"\") will always be true since every string contains an empty string. This should check for the checkmark emoji '✅' as it did previously, or use a more specific success indicator.

Suggested change
message.includes("") ? "bg-green-100 text-green-700" : "bg-red-100 text-red-700"
message.includes("") ? "bg-green-100 text-green-700" : "bg-red-100 text-red-700"

Copilot uses AI. Check for mistakes.
label="Card Number"
required={true}
placeholder="4111 1111 1111 1111"
placeholder="4111 1111 **** ****"

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Card number placeholder changed from full digits to masked format. While this may be intentional for privacy, it could confuse users about the expected input format. Consider using a placeholder that clearly shows the full format is expected, like '1234 5678 9012 3456'.

Suggested change
placeholder="4111 1111 **** ****"
placeholder="1234 5678 9012 3456"

Copilot uses AI. Check for mistakes.
Comment thread docker-compose.yml
npx prisma migrate deploy --schema=prisma/schema.prisma &&
npx prisma generate --schema=prisma/schema.prisma &&
npx prisma db seed --schema=prisma/schema.prisma &&
npx prisma db seedreward --schema=prisma/schema.prisma &&

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The command npx prisma db seedreward is not a valid Prisma CLI command. The correct command is npx prisma db seed. This will cause the setup script to fail.

Copilot uses AI. Check for mistakes.
Comment on lines +11 to 12
import { useMemo } from 'react';

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import useMemo.

Suggested change
import { useMemo } from 'react';

Copilot uses AI. Check for mistakes.
import pic14 from "@/public/three.png";
import pic15 from "@/public/two.png";
import { PointerHighlight } from "../../components/ui/pointer-highlight";
import { ShootingStars } from "@/components/ui/shooting-stars";

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import ShootingStars.

Suggested change
import { ShootingStars } from "@/components/ui/shooting-stars";

Copilot uses AI. Check for mistakes.
import pic15 from "@/public/two.png";
import { PointerHighlight } from "../../components/ui/pointer-highlight";
import { ShootingStars } from "@/components/ui/shooting-stars";
import { StarsBackground } from "@/components/ui/stars-background";

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import StarsBackground.

Suggested change
import { StarsBackground } from "@/components/ui/stars-background";

Copilot uses AI. Check for mistakes.
import { PointerHighlight } from "../../components/ui/pointer-highlight";
import { ShootingStars } from "@/components/ui/shooting-stars";
import { StarsBackground } from "@/components/ui/stars-background";
import { BackgroundBeamsWithCollision } from "@/components/ui/background-beams-with-collision";

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import BackgroundBeamsWithCollision.

Suggested change
import { BackgroundBeamsWithCollision } from "@/components/ui/background-beams-with-collision";

Copilot uses AI. Check for mistakes.
import { DropdownMenu, DropdownMenuContent, DropdownMenuTrigger } from "./dropdown-menu";
import { DropdownMenuItem } from "@radix-ui/react-dropdown-menu";
import { TextHoverEffect } from "../../../../packages/ui/src/text-hover-effect";
import { useTheme } from "next-themes";

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import useTheme.

Copilot uses AI. Check for mistakes.
@VisibleNasir VisibleNasir merged commit 0493628 into main Nov 9, 2025
7 checks passed
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.

2 participants