proxmoxでmemberページが表示できるようにする #231

nose221834 · 2024-02-12T13:47:45Z

対応Issue

resolve #230

概要

proxmoxでmemberページが表示できるようにした

実装内容

nodeのアップデート
next,reactのアップデート
typescriptのアップデート
Dockerfileの編集
DBの編集
Nextのアップデートによるエラーの解消

画面スクリーンショット等

テスト項目

きちんと動作しているか

備考

いまいまいがslackで言ってたことを実装しないとまたメンバーページが見れなくなるので、そこ早急に実装して欲しいです。
あと、アップデートして死ぬほどwarning出てるからこれも解消お願いします。

takumi0616

回答よろしくお願いします！
僕じゃ全然わかんないとこあったので、つっきーにも見てもらおうと思います！
僕の見た限りではApproveです！

takumi0616 · 2024-02-12T14:01:55Z

view/next-project/package.json

@@ -17,10 +17,10 @@
    "highlight.js": "^11.6.0",
    "highlightjs": "^9.16.2",
    "marked": "^4.1.1",
-    "next": "12.0.3",
-    "react": "17.0.2",
+    "next": "^13.5.6",


ここってv14に変更することもできますか？
（やってほしいわけではなくて教えて欲しいことです！）

できる！
ただ、14に変えるともっと色々変えなきゃいけないところ増えると思うからとりあえず13にしたよ
もし14にあげたいなら、今のwaningを直す前に14にしちゃってもいいかもね

takumi0616 · 2024-02-12T14:02:49Z

view/next-project/src/components/common/Header/Header.tsx

+                <Link href='/records'legacyBehavior>
                  <a>Record</a>
                </Link>
              </li>
              <li>
-                <Link href='/curriculums'>
+                <Link href='/curriculums' legacyBehavior>
                  <a>Curriculum</a>
                </Link>
              </li>
              <li>
-                <Link href='/projects'>
+                <Link href='/projects' legacyBehavior>
                  <a>Project</a>
                </Link>
              </li>
              <li>
-                <Link href='/users'>
+                <Link href='/users' legacyBehavior>
                  <a>Member</a>
                </Link>
              </li>
              <li>
-                <Link href='/skills'>
+                <Link href='/skills' legacyBehavior>
                  <a>Skills</a>
                </Link>
              </li>
              <li>
-                <Link href='/categories'>
+                <Link href='/categories' legacyBehavior>


フロント部分も直していただきありがとうございます！

とりあえずエラーが出ないようにしただけだから、また直して欲しい
Linkの中でaタグを使うとエラーがでるんだけど、legacyBehaviorを書けば許された
legacyBehaviorを消してaタグも消すか、これにするかだね

aタグをpタグに直してしまえばlegacyBehaviorは必要なくなると思います！
aタグならaタグ、LinkコンポーネントならLinkコンポーネント単体で使ったほうが良いと思います！

takumi0616 · 2024-02-12T14:05:24Z

api/config/credentials.yml.enc

@@ -1 +1 @@
-eXqxkmurqSMGQLTcA4rpLtPwuuD+MCPVm2PYdaHdhPOT4tdXQSbdOJtX7HVe/6iyHGpQN22aZOLH8OkoTBPPhIBUMc8TD642OceiYDIq0YhtgvCw/MG+o5ncsioyLFw9FEMuohTRpS1ryrovuJtvXAciO58iz7J86lAB8CUzhVrlOJXMDRAX4YFS0+sh3f9xQZho7t2Y0g4k09leaCjWeKdD3QeMlYf0PdmigOre7hQoHhUw3viRjCqm9p4L/75WLL07MQVppS7lF0QLY/dh8L6L76H7aWw1dsUyNMsXIgQoEEEAiKJq9OPJTpSqC0Rdar+d/gROGw5NuSYtPUApiTygbpj3l0uJxzSPpsmrAKJihkxv8TWl2UHVdQW7b3kUFLFnIud8ccRa9UXOOQIG/iO/tiiwcXeSXpD2--naOrfFmuyFZmSsQM--AqntabeEV/PNcpBu/XLXXg==
+f8ko3J21oiEzt+rXqJbVgEJ5XVCMDljL/dqszMhBp0Sg5eVv+OV8nlMldfhGk3FYw2QFhV1gy0rziJ+NQwrBqOQpfveibMbKjwLYDJK5D9lTG3tHifLZoYVbhG4tp6pV8T2o6IxH75bcIKGj8krtpkL+Ijpz4xNUAmslvz/tEXUxtydvrGM+Z/GXqhoNZiVBjwpNWsahleeRvdG8OqnqrzbPxrCKJRoJmNRs537HEZlhwtn8xsUYQ1DBg9Syg3SA0HsC39ouAoKamc2g4k0fHSneSgVuZmFNeh7XwGSMNKpwxZCv4EXF79Lk8/7xDBrdONZ+AEQted7/CdGEV5fvoTu7wc9MHWbleUTClb1cnlbzGMpaKL2bmvOyy3xkIDMVykpT/YJkc+P5p/aNdAvD2ez4eopExX1Bioq2--wdAQtBxw3dn/BJo2--1Cr/bd3OcCoyXUJJ6WN3Ug==


mastaer.keyってなんの文字列なんですか？
credentials.yml.encってどんなこと書いてるファイルなんですか？
（教えていただけるとありがたいです🙇🏻‍♂️）

railsの公開鍵みたいなものだと思ってる。
railsの本番環境はこれとmaster.key（これはsettingsの中にある、こっとが秘密鍵）が必要で、どっちもないといけない。
master.keyは前に変えたんだけどこっちは変え忘れてたから、今回のPRで変更しとした。

nose221834 · 2024-02-12T14:34:13Z

view/next-project/src/components/common/Table/Table.tsx

-      <DataTable
-        columns={props.columns}
-        data={filteredItems}
-        pagination
-        highlightOnHover
-        pointerOnHover
-        onRowClicked={(row) => {
-          if (row.curriculum) router.push(`/${props.route}/${row.curriculum.id}`);
-          else router.push(`/${props.route}/${row.id}`);
-        }}
-        customStyles={CUSTOM_STYLES}
-      />
+      {loader ? (
+        <div>Loading</div>
+      ) : (
+        <DataTable
+          columns={props.columns}
+          data={filteredItems}
+          pagination
+          highlightOnHover
+          pointerOnHover
+          onRowClicked={(row) => {
+            if (row.curriculum) router.push(`/${props.route}/${row.curriculum.id}`);
+            else router.push(`/${props.route}/${row.id}`);
+          }}
+          customStyles={CUSTOM_STYLES}
+        />
+      )}


https://stackoverflow.com/questions/75068071/react-data-table-component-pagination-error-hydration-failed-because-the-in
これと同じエラーが起きてこう書いたけど、無駄なstateな気がしてる

https://github.com/Selman-S/React-Data-Table-Component
このライブラリ、2年前から更新ないやん
リファクタ案件かも～

nose221834 added 15 commits February 11, 2024 21:36

[fix]nodeのアップデート

e484c4f

[fix]react、Nextのアップデート

6c5c6b6

[fix]key propsの追加

0431ba4

[fix] npm audit fix --forceの実行

7f3b84b

[fix]typescriptのアップデート

343a229

[fix]LinkにlegacyBehaviorを付ける

362e426

[fix]DataTableのpaginationで発生したhydrationエラーの修正

08d82cc

[fix]npm install --save-exact --save-dev typescript @types/nodeの実行

0aa4e2f

[fix] npm install typescript @types/nodeの実行

a0846d6

[fix]master.keyの変更に伴って変更

69d347c

[fix] npmのアップデート

c8778af

[fix] user_detailを複数形に変更

e8a390a

[fix]dockerの修正

766b38f

[del] 余分な記述の削除

efbbad9

[fix]user_detailに変更

abf78bd

nose221834 requested review from ryuseifujisaki, YushiroDodo63, KokiWakatsuki and takumi0616 February 12, 2024 13:47

takumi0616 approved these changes Feb 12, 2024

View reviewed changes

nose221834 self-assigned this Feb 12, 2024

nose221834 added bug Something isn't working backend frontend member and removed member backend frontend labels Feb 12, 2024

nose221834 commented Feb 12, 2024

View reviewed changes

KokiWakatsuki approved these changes Feb 14, 2024

View reviewed changes

Merge branch 'develop' into feat/nose/230/update-versions

87c885d

takumi0616 merged commit 0ee50cf into develop Apr 2, 2024

takumi0616 deleted the feat/nose/230/update-versions branch April 2, 2024 14:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

proxmoxでmemberページが表示できるようにする #231

proxmoxでmemberページが表示できるようにする #231

nose221834 commented Feb 12, 2024 •

edited

Loading

takumi0616 left a comment

takumi0616 Feb 12, 2024

nose221834 Feb 12, 2024

takumi0616 Feb 12, 2024

nose221834 Feb 12, 2024

imaimai17468 Feb 12, 2024

takumi0616 Feb 12, 2024 •

edited

Loading

nose221834 Feb 12, 2024

nose221834 Feb 12, 2024

nose221834 Feb 12, 2024

		@@ -1 +1 @@
		eXqxkmurqSMGQLTcA4rpLtPwuuD+MCPVm2PYdaHdhPOT4tdXQSbdOJtX7HVe/6iyHGpQN22aZOLH8OkoTBPPhIBUMc8TD642OceiYDIq0YhtgvCw/MG+o5ncsioyLFw9FEMuohTRpS1ryrovuJtvXAciO58iz7J86lAB8CUzhVrlOJXMDRAX4YFS0+sh3f9xQZho7t2Y0g4k09leaCjWeKdD3QeMlYf0PdmigOre7hQoHhUw3viRjCqm9p4L/75WLL07MQVppS7lF0QLY/dh8L6L76H7aWw1dsUyNMsXIgQoEEEAiKJq9OPJTpSqC0Rdar+d/gROGw5NuSYtPUApiTygbpj3l0uJxzSPpsmrAKJihkxv8TWl2UHVdQW7b3kUFLFnIud8ccRa9UXOOQIG/iO/tiiwcXeSXpD2--naOrfFmuyFZmSsQM--AqntabeEV/PNcpBu/XLXXg==
		f8ko3J21oiEzt+rXqJbVgEJ5XVCMDljL/dqszMhBp0Sg5eVv+OV8nlMldfhGk3FYw2QFhV1gy0rziJ+NQwrBqOQpfveibMbKjwLYDJK5D9lTG3tHifLZoYVbhG4tp6pV8T2o6IxH75bcIKGj8krtpkL+Ijpz4xNUAmslvz/tEXUxtydvrGM+Z/GXqhoNZiVBjwpNWsahleeRvdG8OqnqrzbPxrCKJRoJmNRs537HEZlhwtn8xsUYQ1DBg9Syg3SA0HsC39ouAoKamc2g4k0fHSneSgVuZmFNeh7XwGSMNKpwxZCv4EXF79Lk8/7xDBrdONZ+AEQted7/CdGEV5fvoTu7wc9MHWbleUTClb1cnlbzGMpaKL2bmvOyy3xkIDMVykpT/YJkc+P5p/aNdAvD2ez4eopExX1Bioq2--wdAQtBxw3dn/BJo2--1Cr/bd3OcCoyXUJJ6WN3Ug==

proxmoxでmemberページが表示できるようにする #231

proxmoxでmemberページが表示できるようにする #231

Conversation

nose221834 commented Feb 12, 2024 • edited Loading

対応Issue

概要

実装内容

画面スクリーンショット等

テスト項目

備考

takumi0616 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

takumi0616 Feb 12, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nose221834 commented Feb 12, 2024 •

edited

Loading

takumi0616 Feb 12, 2024 •

edited

Loading