v0.7.1 (#66)

v0.7.1

# Release Notes - TPET v0.7.1

## Fix

- Webhook service page has set required and optional fields correctly.
- fix the error of saving template

.eslintrc.json

@@ -1,3 +1,19 @@
 {
-  "extends": ["next/core-web-vitals", "plugin:prettier/recommended"]
+  "extends": [
+    "next/core-web-vitals",
+    "plugin:prettier/recommended",
+    "plugin:@typescript-eslint/recommended"
+  ],
+  "plugins": ["@typescript-eslint"],
+  "rules": {
+    "@typescript-eslint/naming-convention": [
+      "error",
+      {
+        "selector": "interface",
+        "format": ["PascalCase"]
+      }
+    ],
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/interface-name-prefix": "off"
+  }
 }

.github/PULL_REQUEST_TEMPLATE.md

@@ -3,7 +3,9 @@ Thank you for contributing to the project! 🎉
 -->
 
 <!-- markdownlint-disable MD041 -->
+
 ## Description
+
 <!--
 Provide a brief description of this PR:
 - What issue does this PR address?
@@ -12,9 +14,11 @@ Provide a brief description of this PR:
 -->
 
 ## Type of Change
+
 <!--
 Select the type(s) of changes made (you can select multiple):
 -->
+
 - [ ] 🐛 Bug Fix (fixes an issue)
 - [ ] ✨ New Feature (introduces a new feature)
 - [ ] 💄 UI/UX (improves interface or user experience)
@@ -26,12 +30,14 @@ Select the type(s) of changes made (you can select multiple):
 - [ ] 🔒 Security (addresses security concerns)
 
 ## Related Issues
+
 <!--
 List related issue numbers or links:
 Example: Closes SCRUM-123, Relates to SCRUM-456
 -->
 
 ## Testing
+
 <!--
 Describe how these changes were tested:
 - What tests were performed?
@@ -40,14 +46,17 @@ Describe how these changes were tested:
 -->
 
 ## Screenshots/Videos
+
 <!--
 Provide screenshots or videos to demonstrate the changes (if applicable).
 -->
 
 ## Checklist
+
 <!--
 Ensure the following items are checked before submitting the PR:
 -->
+
 - [ ] I have tested these changes.
 - [ ] I have updated the relevant documentation.
 - [ ] My code adheres to the project’s code standards.
@@ -57,6 +66,7 @@ Ensure the following items are checked before submitting the PR:
 - [ ] All existing tests pass.
 
 ## Additional Notes
+
 <!--
 Add any other relevant notes or considerations:
 -->
@@ -66,4 +76,4 @@ Reminder:
 Before submitting your PR, please review the **Coding Guidelines**: https://aws-educate-tw.notion.site/TPET-Backend-Coding-Guidelines-12e6bfee681780ac89c3cf43854380d4?pvs=4
 -->
 
-<!-- markdownlint-enable MD041 -->
+<!-- markdownlint-enable MD041 -->

.github/workflows/lint-format.yml

@@ -9,25 +9,36 @@ on:
       - "**/*.tsx"
       - "**/*.css"
       - "**/*.scss"
+      - "package.json"
+      - "package-lock.json"
+      - ".eslintrc*"
+      - ".prettierrc*"
 
 jobs:
   lint-format:
     runs-on: ubuntu-latest
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Set up Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: 18.17.0
+          cache: "npm"
 
       - name: Install dependencies
-        run: npm install
+        run: npm ci
 
-      - name: Run Prettier
-        run: npm run format -- --check
+      - name: Type check
+        run: npm run type-check
 
-      - name: Run ESLint
+      - name: Check formatting
+        run: npm run format:check
+
+      - name: Check linting
         run: npm run lint
+
+      - name: Run all checks
+        run: npm run check

package.json

@@ -6,9 +6,13 @@
     "dev": "next dev",
     "build": "next build",
     "start": "next start",
-    "lint": "next lint",
+    "lint": "eslint . --ext .ts,.tsx,.js,.jsx",
+    "lint:fix": "eslint . --ext .ts,.tsx,.js,.jsx --fix",
     "format": "prettier --write .",
-    "fix": "npm run format && npm run lint --fix"
+    "format:check": "prettier --check .",
+    "type-check": "tsc --noEmit",
+    "fix": "npm run type-check && npm run format && npm run lint:fix",
+    "check": "npm run type-check && npm run format:check && npm run lint"
   },
   "dependencies": {
     "@heroicons/react": "^2.1.3",
@@ -43,6 +47,8 @@
     "@types/node": "^20",
     "@types/react": "^18",
     "@types/react-dom": "^18",
+    "@typescript-eslint/eslint-plugin": "^7.2.0",
+    "@typescript-eslint/parser": "^7.2.0",
     "eslint": "^8",
     "eslint-config-next": "14.2.3",
     "eslint-config-prettier": "^9.1.0",

src/app/emailHistory/[runId]/page.tsx

@@ -10,11 +10,11 @@ interface PageProps {
   };
 }
 
-interface rowDataType {
+interface RowDataType {
   [key: string]: string;
 }
 
-interface dataType {
+interface DataType {
   bcc: string[];
   subject: string;
   cc: string[];
@@ -24,7 +24,7 @@ interface dataType {
   sender_local_part: string;
   status: string;
   spreadsheet_file_id: string;
-  row_data: rowDataType;
+  row_data: RowDataType;
   atatachment_file_ids: string[];
   is_generated_certficate: boolean;
   sender_username: string;
@@ -38,7 +38,7 @@ interface dataType {
 }
 
 export default function Page({ params }: PageProps) {
-  const [data, setData] = useState<dataType[]>([]);
+  const [data, setData] = useState<DataType[]>([]);
   const [isLoading, setIsLoading] = useState<boolean>(true);
   const [previousLastEvaluatedKey, setPreviousLastEvaluatedKey] = useState<string | null>(null);
   const [currentLastEvaluatedKey, setCurrentLastEvaluatedKey] = useState<string | null>(null);