opensource
/
dify
mirror of https://github.com/langgenius/dify.git
2
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

simplify pricing modal history handling

This commit is contained in:
yyh 2025-12-26 15:34:50 +08:00
parent 9f3a148732
commit dee2492350
No known key found for this signature in database
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
web/hooks/use-query-params.spec.tsx
View File

@ -119,7 +119,7 @@ describe('useQueryParams hooks', () => {
expect(update.searchParams.has(PRICING_MODAL_QUERY_PARAM)).toBe(false)
})
it('should use replace history when closing', async () => {
it('should use push history when closing', async () => {
// Arrange
const { result, onUrlUpdate } = renderWithAdapter(
() => usePricingModal(),
@ -134,7 +134,7 @@ describe('useQueryParams hooks', () => {
// Assert
await waitFor(() => expect(onUrlUpdate).toHaveBeenCalled())
const update = onUrlUpdate.mock.calls[onUrlUpdate.mock.calls.length - 1][0]
expect(update.options.history).toBe('replace')
expect(update.options.history).toBe('push')
})
it('should respect explicit history options when provided', async () => {

7
web/hooks/use-query-params.ts
View File

@ -34,6 +34,8 @@ const parseAsPricingModal = createParser<boolean>({
parse: value => (value === PRICING_MODAL_QUERY_VALUE ? true : null),
serialize: value => (value ? PRICING_MODAL_QUERY_VALUE : ''),
})
.withDefault(false)
.withOptions({ history: 'push' })
/**
* Hook to manage pricing modal state via URL
@ -47,13 +49,12 @@ const parseAsPricingModal = createParser<boolean>({
export function usePricingModal() {
const [isOpen, setIsOpenState] = useQueryState(
PRICING_MODAL_QUERY_PARAM,
parseAsPricingModal.withDefault(false),
parseAsPricingModal,
)
const setIsOpen = useCallback(
(open: boolean, options?: Options) => {
const history = options?.history ?? (open ? 'push' : 'replace')
setIsOpenState(open ? true : null, { ...options, history })
setIsOpenState(open, options)
},
[setIsOpenState],
)