fix(ui): close copy locale modal after locale is changed (#10096)

r1tsuu · web-flow · commit 59fc9d094e2d · 2024-12-20T08:29:53.000-05:00
Ensures we close the modal _only_ after locale is changed. This caused our localization e2e's to flake: ![image](https://github.com/user-attachments/assets/41205afe-0e45-499b-9aa6-07734a7f26fc)
diff --git a/packages/ui/src/elements/CopyLocaleData/index.tsx b/packages/ui/src/elements/CopyLocaleData/index.tsx
@@ -77,10 +77,10 @@ export const CopyLocaleData: React.FC = () => {
         })
 
         setCopying(false)
-        toggleModal(drawerSlug)
         router.push(
           `${serverURL}${admin}/${collectionSlug ? `collections/${collectionSlug}/${id}` : `globals/${globalSlug}`}?locale=${to}`,
         )
+        toggleModal(drawerSlug)
       } catch (error) {
         toast.error(error.message)
       }

Original file line number	Diff line number	Diff line change
`@@ -77,10 +77,10 @@ export const CopyLocaleData: React.FC = () => {`
`77`	`77`	`})`
`78`	`78`
`79`	`79`	`setCopying(false)`
`80`		`- toggleModal(drawerSlug)`
`81`	`80`	`router.push(`
`82`	`81`	`${serverURL}${admin}/${collectionSlug ? `collections/${collectionSlug}/${id}` : `globals/${globalSlug}`}?locale=${to}`,
`83`	`82`	`)`
	`83`	`+ toggleModal(drawerSlug)`
`84`	`84`	`} catch (error) {`
`85`	`85`	`toast.error(error.message)`
`86`	`86`	`}`