feat: minor bugs (#1418)

* fix: minor bugs

* feat: reset row selection in user management and feedback tables

* fix: update dependencies in useEffect for field setting component

* fix: update next.config file extension in Dockerfile

* refactor: simplify query state management and improve filter handling

* fix: ensure setOperator is awaited in feedback and issue query converters

* feat: add linkify and linkify-react for enhanced text linking in feedback and sheet detail views

* fix: update Tooltip behavior to open when no projectId is selected

* feat: add operator prop to TableFilterPopover and update related components

* feat: add operator prop to TableFilterPopover in user management and member settings

---------

Co-authored-by: jeehoon.choi <jeehoon.choi@linecorp.com>

Author: chiol

apps/cli/bin/auf-cli.ts

@@ -106,11 +106,7 @@ program
     const tomlContent = fs.readFileSync(destinationConfigPath, 'utf-8');
     const tomlConfig = load(tomlContent) as TomlConfig;
 
-    const webEnvVars = [
-      'NEXT_PUBLIC_API_BASE_URL',
-      'NEXT_PUBLIC_MAX_DAYS',
-      'SESSION_PASSWORD',
-    ];
+    const webEnvVars = ['NEXT_PUBLIC_API_BASE_URL', 'NEXT_PUBLIC_MAX_DAYS'];
 
     const apiEnvVars = [
       'JWT_SECRET',

apps/cli/config.toml

@@ -18,4 +18,3 @@ MASTER_API_KEY="MASTER_API_KEY"
 [web]
 NEXT_PUBLIC_API_BASE_URL="http://localhost:4000"
 NEXT_PUBLIC_MAX_DAYS=90
-SESSION_PASSWORD="sessionpasswordsessionpasswordsessionpassword"

apps/cli/docker-compose.template.yml

@@ -10,7 +10,6 @@ services:
     environment:
       - NEXT_PUBLIC_API_BASE_URL=${NEXT_PUBLIC_API_BASE_URL}
       - NEXT_PUBLIC_MAX_DAYS=${NEXT_PUBLIC_MAX_DAYS}
-      - SESSION_PASSWORD=${SESSION_PASSWORD}
     depends_on:
       - api
 

apps/web/README.md

@@ -63,10 +63,9 @@ pnpm build
 
 ### Required Environment Variables
 
-| Environment              | Description                                             | Default Value                                |
-| ------------------------ | ------------------------------------------------------- | -------------------------------------------- |
-| NEXT_PUBLIC_API_BASE_URL | api base url in client side (ex. http://localhost:4000) |                                              |
-| SESSION_PASSWORD         | session password                                        | complex_password_at_least_32_characters_long |
+| Environment              | Description                                             | Default Value |
+| ------------------------ | ------------------------------------------------------- | ------------- |
+| NEXT_PUBLIC_API_BASE_URL | api base url in client side (ex. http://localhost:4000) |               |
 
 ### Optional Environment Variables
 

apps/web/package.json

@@ -58,6 +58,8 @@
     "i18next": "^24.0.0",
     "immer": "^10.0.4",
     "jwt-decode": "^4.0.0",
+    "linkify": "^0.2.1",
+    "linkify-react": "^4.2.0",
     "next": "^15.0.0",
     "next-i18next": "^15.3.0",
     "next-themes": "^0.4.3",

apps/web/src/entities/feedback/lib/use-feedback-query-converter.ts

@@ -14,9 +14,9 @@
  * under the License.
  */
 
-import { useCallback, useMemo, useState } from 'react';
+import { useCallback, useMemo } from 'react';
 import dayjs from 'dayjs';
-import { createParser, useQueryState } from 'nuqs';
+import { createParser, parseAsStringLiteral, useQueryState } from 'nuqs';
 
 import type {
   DateRangeType,
@@ -26,7 +26,7 @@ import type {
   TableFilterFieldFotmat,
   TableFilterOperator,
 } from '@/shared';
-import { client } from '@/shared';
+import { client, TableFilterOperators } from '@/shared';
 
 import { env } from '@/env';
 
@@ -86,7 +86,11 @@ const useFeedbackQueryConverter = (input: {
 }) => {
   const { projectId, filterFields } = input;
 
-  const [operator, setOperator] = useState<TableFilterOperator>('AND');
+  const [operator, setOperator] = useQueryState<TableFilterOperator>(
+    'operator',
+    parseAsStringLiteral(TableFilterOperators).withDefault('AND'),
+  );
+
   const [queries, setQueries] = useQueryState<SearchQuery[]>(
     'queries',
     createParser({
@@ -122,7 +126,7 @@ const useFeedbackQueryConverter = (input: {
 
   const onChangeDateRange = useCallback(async (value: DateRangeType) => {
     if (!value) return;
-    console.log('value: ', value);
+
     await setDefaultQueries([
       {
         key: 'createdAt',
@@ -162,11 +166,11 @@ const useFeedbackQueryConverter = (input: {
           { key: 'createdAt', value: DEFAULT_DATE_RANGE, condition: 'BETWEEN' },
         ]);
         await setQueries([]);
-        setOperator('AND');
+        await setOperator('AND');
         return;
       }
 
-      setOperator(operator);
+      await setOperator(operator);
       await setQueries(result);
     },
     [dateRange],

apps/web/src/entities/feedback/ui/feedback-cell.tsx

@@ -15,15 +15,11 @@
  */
 import { memo } from 'react';
 import dayjs from 'dayjs';
+import Linkify from 'linkify-react';
 
 import { Badge } from '@ufb/react';
 
-import {
-  DATE_TIME_FORMAT,
-  ExpandableText,
-  ImagePreviewButton,
-  linkify,
-} from '@/shared';
+import { DATE_TIME_FORMAT, ExpandableText, ImagePreviewButton } from '@/shared';
 import type { FieldInfo } from '@/entities/field';
 
 interface IProps {
@@ -73,7 +69,20 @@ const FeedbackCell: React.FC<IProps> = memo((props) => {
           {field.format === 'images' && (
             <ImagePreviewButton urls={value as string[]} />
           )}
-          {field.format === 'text' && linkify(value as string)}
+          {field.format === 'text' && (
+            <Linkify
+              options={{
+                className: 'text-blue-500 underline',
+                target: '_blank',
+                rel: 'noopener noreferrer',
+                attributes: {
+                  onClick: (e: React.MouseEvent) => e.stopPropagation(),
+                },
+              }}
+            >
+              {value as string | undefined}
+            </Linkify>
+          )}
           {field.format === 'keyword' && value}
           {field.format === 'number' && value}
         </>

apps/web/src/entities/user/ui/user-management-table.ui.tsx

@@ -145,6 +145,7 @@ const UserManagementTable: React.FC<IProps> = ({ createButton }) => {
     setRows(userData?.items ?? []);
     setPageCount(userData?.meta.totalPages ?? 0);
     setRowCount(userData?.meta.totalItems ?? 0);
+    table.resetRowSelection();
   }, [userData, pagination, isLoading]);
 
   useEffect(() => {
@@ -189,6 +190,7 @@ const UserManagementTable: React.FC<IProps> = ({ createButton }) => {
       <div className="flex items-center justify-between">
         <div className="flex items-center gap-2">
           <TableFilterPopover
+            operator={operator}
             filterFields={[
               {
                 key: 'email',

apps/web/src/entities/user/ui/user-profile-setting.ui.tsx

@@ -45,9 +45,10 @@ const UserProfileSetting = () => {
     resolver: zodResolver(userProfileSchema),
   });
 
-  const { data, refetch, isRefetching } = useOAIQuery({
+  const { data, refetch } = useOAIQuery({
     path: '/api/admin/users/{id}',
     variables: { id: user?.id ?? 0 },
+    queryOptions: { refetchOnWindowFocus: false },
   });
 
   const { mutate: updateProfile, isPending: isPendingUpdateProfile } =
@@ -77,7 +78,7 @@ const UserProfileSetting = () => {
 
   useEffect(() => {
     methods.reset(data);
-  }, [data, isRefetching]);
+  }, [data]);
 
   useWarnIfUnsavedChanges(methods.formState.isDirty);
 

apps/web/src/entities/user/user.schema.ts

@@ -69,10 +69,15 @@ export const changePasswordSchema = z
       path: ['confirmNewPassword'],
     },
   )
+  .refine(({ newPassword }) => /[A-Za-z]/.test(newPassword), {
+    message: 'must contain at least one letter.',
+    path: ['newPassword'],
+  })
   .refine(
-    ({ newPassword }) => /[a-zA-Z!@#$%^&*(),.?":{}|<>]/.test(newPassword),
+    ({ newPassword }) =>
+      /[!@#$%^&*()_+\-=[\]{};':"\\|,.<>/?`~]/.test(newPassword),
     {
-      message: 'must contain at least one letter or special character',
+      message: 'must contain at least one special character.',
       path: ['newPassword'],
     },
   )
@@ -92,10 +97,17 @@ export const invitedUserSignupSchema = z
     message: 'must equal Password',
     path: ['confirmPassword'],
   })
-  .refine(({ password }) => /[a-zA-Z!@#$%^&*(),.?":{}|<>]/.test(password), {
-    message: 'must contain at least one letter or special character',
+  .refine(({ password }) => /[A-Za-z]/.test(password), {
+    message: 'must contain at least one letter.',
     path: ['password'],
   })
+  .refine(
+    ({ password }) => /[!@#$%^&*()_+\-=[\]{};':"\\|,.<>/?`~]/.test(password),
+    {
+      message: 'Password must contain at least one special character.',
+      path: ['password'],
+    },
+  )
   .refine(({ password }) => !/(.)\1/.test(password), {
     message: 'must not contain consecutive identical characters',
     path: ['password'],