Skip to content

Commit

Permalink
chore: pr feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
JeapfromtheLake committed Jan 30, 2024
1 parent 63e0f67 commit 957280b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ export const LocaleSelect = ({ onChange, isDisabled, value, isLoadingValue }: Pr
value={value}
onChange={handleChange}
>
{(allLocales || []).map((l, index) => (
<SingleSelectOption value={l.code} key={index}>
{(allLocales || []).map((l) => (
<SingleSelectOption value={l.code} key={l.code}>
{l.code.toLocaleUpperCase()}
</SingleSelectOption>
))}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ interface IInternalLinkModalProps
attribute?: IInternalLinkAttribute;
toggleModal: () => void;
closeModal: () => void;
alwaysUseTitle?: boolean;
overwriteNoTitle?: boolean;
}

const InternalLinkModal = ({
Expand All @@ -25,7 +25,7 @@ const InternalLinkModal = ({
errors,
setErrors,
attribute,
alwaysUseTitle,
overwriteNoTitle,
toggleModal = () => {
console.warn('Modal toggle function not set');
},
Expand All @@ -37,7 +37,7 @@ const InternalLinkModal = ({
const { data: pluginConfig, isLoading: isLoadingConfig } = useGetConfig({});

const shouldShowTitle =
alwaysUseTitle ??
overwriteNoTitle ??
(!isLoadingConfig &&
(typeof attribute?.options?.noTitle === 'boolean'
? !attribute?.options?.noTitle
Expand Down

0 comments on commit 957280b

Please sign in to comment.