mirrors/cinny
1
0
Fork 0
mirror of https://github.com/cinnyapp/cinny.git synced 2025-11-04 22:40:29 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix profile field comparison

Browse source
This commit is contained in:
Ginger 2025-09-20 14:40:13 -04:00
parent 79b37e177b
commit cfee62ffe6
No known key found for this signature in database
2 changed files with 54 additions and 29 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/app/features/settings/account/Profile.tsx
View file

@ -60,7 +60,7 @@ function IdentityProviderSettings({ authMetadata }: { authMetadata: ValidatedAut
</Button>
}
>
<Text size="T200">Change profile settings in your homeserver's account dashboard.</Text>
<Text size="T200">Change profile settings in your homeserver&apos;s account dashboard.</Text>
</SettingTile>
</CutoutCard>
);

55
src/app/features/settings/account/fields/ProfileFieldContext.tsx
View file

@ -1,26 +1,43 @@
import React, { FunctionComponent, ReactNode, useCallback, useEffect, useMemo, useState } from 'react';
import React, {
FunctionComponent,
ReactNode,
useCallback,
useEffect,
useMemo,
useState,
} from 'react';
import { ExtendedProfile } from '../../../../hooks/useExtendedProfile';
type ExtendedProfileKeys = keyof {
[Property in keyof ExtendedProfile as string extends Property ? never : Property]: ExtendedProfile[Property]
}
[Property in keyof ExtendedProfile as string extends Property
? never
: Property]: ExtendedProfile[Property];
};
type ProfileFieldElementRawProps<V, C> = {
defaultValue: V,
value: V,
setValue: (value: V) => void,
} & C
defaultValue: V;
value: V;
setValue: (value: V) => void;
} & C;
export type ProfileFieldElementProps<K extends ExtendedProfileKeys, C> = ProfileFieldElementRawProps<ExtendedProfile[K], C>;
export type ProfileFieldElementProps<
K extends ExtendedProfileKeys,
C
> = ProfileFieldElementRawProps<ExtendedProfile[K], C>;
type ProfileFieldElements<C> = {
[Property in ExtendedProfileKeys]?: FunctionComponent<ProfileFieldElementProps<Property, C>>;
}
};
type ProfileFieldContextProps<C> = {
fieldDefaults: ExtendedProfile;
fieldElements: ProfileFieldElements<C>;
children: (reset: () => void, hasChanges: boolean, fields: ExtendedProfile, fieldElements: ReactNode) => ReactNode;
children: (
reset: () => void,
hasChanges: boolean,
fields: ExtendedProfile,
fieldElements: ReactNode
) => ReactNode;
context: C;
};
@ -28,7 +45,7 @@ export function ProfileFieldContext<C>({
fieldDefaults,
fieldElements: fieldElementConstructors,
children,
context
context,
}: ProfileFieldContextProps<C>): ReactNode {
const [fields, setFields] = useState<ExtendedProfile>(fieldDefaults);
@ -37,7 +54,7 @@ export function ProfileFieldContext<C>({
}, [fieldDefaults]);
useEffect(() => {
reset()
reset();
}, [reset]);
const setField = useCallback(
@ -51,11 +68,17 @@ export function ProfileFieldContext<C>({
);
const hasChanges = useMemo(
() => Object.entries(fields).find(([key, value]) => fieldDefaults[key as keyof ExtendedProfile] !== value) !== undefined,
() =>
Object.entries(fields).find(
([key, value]) =>
// this is a hack but ExtendedProfile is always valid JSON anyway
JSON.stringify(fieldDefaults[key as keyof ExtendedProfile]) !== JSON.stringify(value)
) !== undefined,
[fields, fieldDefaults]
);
const createElement = useCallback(<K extends ExtendedProfileKeys>(key: K, element: ProfileFieldElements<C>[K]) => {
const createElement = useCallback(
<K extends ExtendedProfileKeys>(key: K, element: ProfileFieldElements<C>[K]) => {
const props: ProfileFieldElementRawProps<ExtendedProfile[K], C> = {
...context,
defaultValue: fieldDefaults[key],
@ -67,7 +90,9 @@ export function ProfileFieldContext<C>({
return React.createElement(element, props);
}
return undefined;
}, [context, fieldDefaults, fields, setField]);
},
[context, fieldDefaults, fields, setField]
);
const fieldElements = Object.entries(fieldElementConstructors).map(([key, element]) =>
// @ts-expect-error TypeScript doesn't quite understand the magic going on here