fix: exclude JP from targetable country list

Resolves brave/ads-serve#4555

While I'm there, sort the list by name rather than code (because code just looks odd).

And fix one of the react warnings.

src/components/Location/LocationPicker.tsx

@@ -8,16 +8,23 @@ import { useLingui } from "@lingui/react";
 import { msg, Trans } from "@lingui/macro";
 import { useQuery } from "@apollo/client";
 import { useIsEdit } from "@/form/FormikHelpers";
+import { useMemo } from "react";
 
 const icon = <CheckBoxOutlineBlankIcon fontSize="small" />;
 const checkedIcon = <CheckBoxIcon fontSize="small" />;
 
 export const LocationPicker = () => {
   const { isDraft } = useIsEdit();
   const { data } = useQuery(ActiveGeocodesDocument, { skip: !isDraft });
-  const sorted = _.sortBy(
-    (data?.geocodes ?? []).filter((c) => c.code !== "OTHER"),
-    "code",
+  const sorted = useMemo(
+    () =>
+      _.sortBy(
+        (data?.geocodes ?? []).filter(
+          (c) => c.code !== "OTHER" && c.code !== "JP",
+        ),
+        "name",
+      ),
+    [data],
   );
   const [formProps, meta, helper] = useField<GeocodeInput[]>("geoTargets");
   const errorMessage = meta.error;

src/components/Navigation/AccountMenu.tsx

@@ -121,6 +121,7 @@ export function AccountMenu() {
         )}
         {nonCurrentAdvertisers.map((a, idx) => (
           <MenuItem
+            key={a.id}
             onClick={() => {
               setActiveAdvertiser(a.id);
               setSessionUser({ ...user, id: user.userId, advertisers } as any);