refactor: replace getCreditBalanceAction with useCredits hook in credits-related components

This commit is contained in:
javayhu 2025-07-11 01:10:43 +08:00
parent 5cb8b0048d
commit e6663b013d
4 changed files with 105 additions and 74 deletions

View File

@ -1,35 +1,26 @@
'use client'; 'use client';
import { getCreditBalanceAction } from '@/actions/get-credit-balance';
import { Button } from '@/components/ui/button'; import { Button } from '@/components/ui/button';
import { useCredits } from '@/hooks/use-credits';
import { useLocaleRouter } from '@/i18n/navigation'; import { useLocaleRouter } from '@/i18n/navigation';
import { Routes } from '@/routes'; import { Routes } from '@/routes';
import { useCreditTransactionStore } from '@/stores/transaction-store'; import { useCreditTransactionStore } from '@/stores/transaction-store';
import { CoinsIcon, Loader2Icon } from 'lucide-react'; import { CoinsIcon, Loader2Icon } from 'lucide-react';
import { useEffect, useState } from 'react'; import { useEffect } from 'react';
export function CreditsBalanceButton() { export function CreditsBalanceButton() {
const router = useLocaleRouter(); const router = useLocaleRouter();
const { refreshTrigger } = useCreditTransactionStore(); const { refreshTrigger } = useCreditTransactionStore();
const [credits, setCredits] = useState<number>(0);
const [loading, setLoading] = useState(true); // Use the new useCredits hook
const { balance, isLoading, refresh } = useCredits();
useEffect(() => { useEffect(() => {
const fetchCredits = async () => { // Refresh credits when transaction refresh is triggered
try { if (refreshTrigger) {
const result = await getCreditBalanceAction(); refresh();
if (result?.data?.success && result.data.credits !== undefined) { }
setCredits(result.data.credits); }, [refreshTrigger, refresh]);
}
} catch (error) {
console.error('CreditsBalanceButton, fetch credits error:', error);
} finally {
setLoading(false);
}
};
fetchCredits();
}, [refreshTrigger]);
const handleClick = () => { const handleClick = () => {
router.push(Routes.SettingsCredits); router.push(Routes.SettingsCredits);
@ -44,10 +35,10 @@ export function CreditsBalanceButton() {
> >
<CoinsIcon className="h-4 w-4" /> <CoinsIcon className="h-4 w-4" />
<span className=""> <span className="">
{loading ? ( {isLoading ? (
<Loader2Icon className="h-4 w-4 animate-spin" /> <Loader2Icon className="h-4 w-4 animate-spin" />
) : ( ) : (
credits.toLocaleString() balance.toLocaleString()
)} )}
</span> </span>
</Button> </Button>

View File

@ -1,36 +1,27 @@
'use client'; 'use client';
import { getCreditBalanceAction } from '@/actions/get-credit-balance'; import { useCredits } from '@/hooks/use-credits';
import { useLocaleRouter } from '@/i18n/navigation'; import { useLocaleRouter } from '@/i18n/navigation';
import { Routes } from '@/routes'; import { Routes } from '@/routes';
import { useCreditTransactionStore } from '@/stores/transaction-store'; import { useCreditTransactionStore } from '@/stores/transaction-store';
import { CoinsIcon, Loader2Icon } from 'lucide-react'; import { CoinsIcon, Loader2Icon } from 'lucide-react';
import { useTranslations } from 'next-intl'; import { useTranslations } from 'next-intl';
import { useEffect, useState } from 'react'; import { useEffect } from 'react';
export function CreditsBalanceMenu() { export function CreditsBalanceMenu() {
const t = useTranslations('Marketing.avatar'); const t = useTranslations('Marketing.avatar');
const router = useLocaleRouter(); const router = useLocaleRouter();
const { refreshTrigger } = useCreditTransactionStore(); const { refreshTrigger } = useCreditTransactionStore();
const [credits, setCredits] = useState<number>(0);
const [loading, setLoading] = useState(true); // Use the new useCredits hook
const { balance, isLoading, refresh } = useCredits();
useEffect(() => { useEffect(() => {
const fetchCredits = async () => { // Refresh credits when transaction refresh is triggered
try { if (refreshTrigger) {
const result = await getCreditBalanceAction(); refresh();
if (result?.data?.success && result.data.credits !== undefined) { }
setCredits(result.data.credits); }, [refreshTrigger, refresh]);
}
} catch (error) {
console.error('CreditsBalanceMenu, fetch credits error:', error);
} finally {
setLoading(false);
}
};
fetchCredits();
}, [refreshTrigger]);
const handleClick = () => { const handleClick = () => {
router.push(Routes.SettingsCredits); router.push(Routes.SettingsCredits);
@ -47,10 +38,10 @@ export function CreditsBalanceMenu() {
</div> </div>
<div className="flex items-center"> <div className="flex items-center">
<p className="text-sm font-medium"> <p className="text-sm font-medium">
{loading ? ( {isLoading ? (
<Loader2Icon className="h-4 w-4 animate-spin" /> <Loader2Icon className="h-4 w-4 animate-spin" />
) : ( ) : (
credits.toLocaleString() balance.toLocaleString()
)} )}
</p> </p>
</div> </div>

View File

@ -1,6 +1,5 @@
'use client'; 'use client';
import { getCreditBalanceAction } from '@/actions/get-credit-balance';
import { Badge } from '@/components/ui/badge'; import { Badge } from '@/components/ui/badge';
import { import {
Card, Card,
@ -10,6 +9,7 @@ import {
CardTitle, CardTitle,
} from '@/components/ui/card'; } from '@/components/ui/card';
import { getCreditPackages } from '@/config/credits-config'; import { getCreditPackages } from '@/config/credits-config';
import { useCredits } from '@/hooks/use-credits';
import { useCurrentUser } from '@/hooks/use-current-user'; import { useCurrentUser } from '@/hooks/use-current-user';
import { useLocaleRouter } from '@/i18n/navigation'; import { useLocaleRouter } from '@/i18n/navigation';
import { formatPrice } from '@/lib/formatter'; import { formatPrice } from '@/lib/formatter';
@ -19,7 +19,7 @@ import { useCreditTransactionStore } from '@/stores/transaction-store';
import { CircleCheckBigIcon, CoinsIcon, Loader2Icon } from 'lucide-react'; import { CircleCheckBigIcon, CoinsIcon, Loader2Icon } from 'lucide-react';
import { useTranslations } from 'next-intl'; import { useTranslations } from 'next-intl';
import { useSearchParams } from 'next/navigation'; import { useSearchParams } from 'next/navigation';
import { useEffect, useState } from 'react'; import { useEffect } from 'react';
import { toast } from 'sonner'; import { toast } from 'sonner';
import { CreditCheckoutButton } from './credit-checkout-button'; import { CreditCheckoutButton } from './credit-checkout-button';
@ -29,38 +29,21 @@ import { CreditCheckoutButton } from './credit-checkout-button';
*/ */
export function CreditPackages() { export function CreditPackages() {
const t = useTranslations('Dashboard.settings.credits.packages'); const t = useTranslations('Dashboard.settings.credits.packages');
const [loadingCredits, setLoadingCredits] = useState(true);
const [credits, setCredits] = useState<number | null>(null);
const { refreshTrigger, triggerRefresh } = useCreditTransactionStore();
const currentUser = useCurrentUser();
const searchParams = useSearchParams(); const searchParams = useSearchParams();
const localeRouter = useLocaleRouter(); const localeRouter = useLocaleRouter();
// Use the new useCredits hook
const { balance, isLoading, refresh } = useCredits();
const { refreshTrigger, triggerRefresh } = useCreditTransactionStore();
// Get current user
const currentUser = useCurrentUser();
// show only enabled packages // show only enabled packages
const creditPackages = Object.values(getCreditPackages()).filter( const creditPackages = Object.values(getCreditPackages()).filter(
(pkg) => !pkg.disabled && pkg.price.priceId (pkg) => !pkg.disabled && pkg.price.priceId
); );
const fetchCredits = async () => {
try {
setLoadingCredits(true);
const result = await getCreditBalanceAction();
if (result?.data?.success) {
console.log('CreditPackages, fetched credits:', result.data.credits);
setCredits(result.data.credits || 0);
} else {
const errorMessage = result?.data?.error || t('failedToFetchCredits');
console.error('CreditPackages, failed to fetch credits:', errorMessage);
toast.error(errorMessage);
}
} catch (error) {
console.error('CreditPackages, failed to fetch credits:', error);
toast.error(t('failedToFetchCredits'));
} finally {
setLoadingCredits(false);
}
};
// Check for payment success and show success message // Check for payment success and show success message
useEffect(() => { useEffect(() => {
const sessionId = searchParams.get('session_id'); const sessionId = searchParams.get('session_id');
@ -72,6 +55,8 @@ export function CreditPackages() {
// Refresh credits data to show updated balance // Refresh credits data to show updated balance
triggerRefresh(); triggerRefresh();
// Also refresh the credits store
refresh();
// Clean up URL parameters // Clean up URL parameters
const url = new URL(window.location.href); const url = new URL(window.location.href);
@ -79,12 +64,14 @@ export function CreditPackages() {
// Use Routes.SettingsCredits + url.search to properly handle locale routing // Use Routes.SettingsCredits + url.search to properly handle locale routing
localeRouter.replace(Routes.SettingsCredits + url.search); localeRouter.replace(Routes.SettingsCredits + url.search);
} }
}, [searchParams, localeRouter]); }, [searchParams, localeRouter, refresh, triggerRefresh, t]);
// Initial fetch and listen for transaction updates // Listen for transaction updates and refresh credits
useEffect(() => { useEffect(() => {
fetchCredits(); if (refreshTrigger) {
}, [refreshTrigger]); refresh();
}
}, [refreshTrigger, refresh]);
return ( return (
<div className="space-y-6"> <div className="space-y-6">
@ -99,11 +86,11 @@ export function CreditPackages() {
<div className="flex items-center space-x-2"> <div className="flex items-center space-x-2">
{/* <CoinsIcon className="h-4 w-4 text-muted-foreground" /> */} {/* <CoinsIcon className="h-4 w-4 text-muted-foreground" /> */}
<div> <div>
{loadingCredits ? ( {isLoading ? (
<Loader2Icon className="h-8 w-8 animate-spin" /> <Loader2Icon className="h-8 w-8 animate-spin" />
) : ( ) : (
<div className="text-2xl font-bold"> <div className="text-2xl font-bold">
{credits?.toLocaleString() || 0} {balance.toLocaleString()}
</div> </div>
)} )}
</div> </div>

62
src/hooks/use-credits.ts Normal file
View File

@ -0,0 +1,62 @@
import { authClient } from '@/lib/auth-client';
import { useCreditsStore } from '@/stores/credits-store';
import { useEffect } from 'react';
/**
* Hook for accessing and managing credits state
*
* This hook provides access to the credits state and methods to manage it.
* It also automatically fetches credits information when the user changes.
*/
export function useCredits() {
const {
balance,
isLoading,
error,
fetchCredits,
consumeCredits,
refreshCredits,
updateBalanceOptimistically,
} = useCreditsStore();
const { data: session } = authClient.useSession();
useEffect(() => {
const currentUser = session?.user;
// Fetch credits data whenever the user session changes
if (currentUser) {
console.log('fetching credits info for user', currentUser.id);
fetchCredits(currentUser);
}
}, [session, fetchCredits]);
return {
// State
balance,
isLoading,
error,
// Methods
consumeCredits,
updateBalanceOptimistically,
// Utility methods
refetch: () => {
const currentUser = session?.user;
if (currentUser) {
console.log('refetching credits info for user', currentUser.id);
fetchCredits(currentUser);
}
},
refresh: () => {
const currentUser = session?.user;
if (currentUser) {
console.log('refreshing credits info for user', currentUser.id);
refreshCredits(currentUser);
}
},
// Helper methods
hasEnoughCredits: (amount: number) => balance >= amount,
};
}