feat: workspace update name and logo (#553)

* feat: workspace update name and logo

* fix: remove logs

* fix: disable warning until refacto

* Fix text

---------

Co-authored-by: Emilien <emilien.chauvet.enpc@gmail.com>
This commit is contained in:
Jérémy M
2023-07-10 20:23:58 +02:00
committed by GitHub
parent a2da3a5f09
commit c9292365c0
7 changed files with 192 additions and 8 deletions

View File

@ -56,7 +56,6 @@ export function SettingsNavbar() {
label="Members"
to="/settings/workspace-members"
icon={<IconUsers size={theme.icon.size.md} />}
soon={false}
active={
!!useMatch({
path: useResolvedPath('/settings/workspace-members').pathname,
@ -68,7 +67,6 @@ export function SettingsNavbar() {
label="General"
to="/settings/workspace"
icon={<IconSettings size={theme.icon.size.md} />}
soon={true}
active={
!!useMatch({
path: useResolvedPath('/settings/workspace').pathname,

View File

@ -0,0 +1,82 @@
import { useCallback, useEffect, useState } from 'react';
import { getOperationName } from '@apollo/client/utilities';
import styled from '@emotion/styled';
import debounce from 'lodash.debounce';
import { useRecoilState } from 'recoil';
import { currentUserState } from '@/auth/states/currentUserState';
import { TextInput } from '@/ui/components/inputs/TextInput';
import { GET_CURRENT_USER } from '@/users/queries';
import { useUpdateWorkspaceMutation } from '~/generated/graphql';
const StyledComboInputContainer = styled.div`
display: flex;
flex-direction: row;
> * + * {
margin-left: ${({ theme }) => theme.spacing(4)};
}
`;
type OwnProps = {
autoSave?: boolean;
onNameUpdate?: (name: string) => void;
};
export function NameField({ autoSave = true, onNameUpdate }: OwnProps) {
const [currentUser] = useRecoilState(currentUserState);
const workspace = currentUser?.workspaceMember?.workspace;
const [displayName, setDisplayName] = useState(workspace?.displayName ?? '');
const [updateWorkspace] = useUpdateWorkspaceMutation();
// TODO: Enhance this with react-hook-form (https://www.react-hook-form.com)
// eslint-disable-next-line react-hooks/exhaustive-deps
const debouncedUpdate = useCallback(
debounce(async (name: string) => {
if (onNameUpdate) {
onNameUpdate(displayName);
}
if (!autoSave || !name) {
return;
}
try {
const { data, errors } = await updateWorkspace({
variables: {
data: {
displayName: {
set: name,
},
},
},
refetchQueries: [getOperationName(GET_CURRENT_USER) ?? ''],
awaitRefetchQueries: true,
});
if (errors || !data?.updateWorkspace) {
throw errors;
}
} catch (error) {
console.error(error);
}
}, 500),
[updateWorkspace],
);
useEffect(() => {
debouncedUpdate(displayName);
return debouncedUpdate.cancel;
}, [debouncedUpdate, displayName]);
return (
<StyledComboInputContainer>
<TextInput
label="Name"
value={displayName}
onChange={setDisplayName}
placeholder="Apple"
fullWidth
/>
</StyledComboInputContainer>
);
}

View File

@ -1,4 +1,4 @@
import { ChangeEvent, useState } from 'react';
import { ChangeEvent } from 'react';
import styled from '@emotion/styled';
type OwnProps = Omit<
@ -52,16 +52,13 @@ export function TextInput({
fullWidth,
...props
}: OwnProps): JSX.Element {
const [internalValue, setInternalValue] = useState(value);
return (
<StyledContainer>
{label && <StyledLabel>{label}</StyledLabel>}
<StyledInput
fullWidth={fullWidth ?? false}
value={internalValue}
value={value}
onChange={(event: ChangeEvent<HTMLInputElement>) => {
setInternalValue(event.target.value);
if (onChange) {
onChange(event.target.value);
}

View File

@ -1,3 +1,4 @@
import { useState } from 'react';
import { expect } from '@storybook/jest';
import { jest } from '@storybook/jest';
import type { Meta, StoryObj } from '@storybook/react';
@ -17,9 +18,22 @@ type Story = StoryObj<typeof TextInput>;
const changeJestFn = jest.fn();
function FakeTextInput({ onChange }: any) {
const [value, setValue] = useState<string>('A good value ');
return (
<TextInput
value={value}
onChange={(text) => {
setValue(text);
onChange(text);
}}
/>
);
}
export const Default: Story = {
render: getRenderWrapperForComponent(
<TextInput value="A good value " onChange={changeJestFn} />,
<FakeTextInput onChange={changeJestFn} />,
),
play: async ({ canvasElement }) => {
const canvas = within(canvasElement);