fix: create public room no password

[bxbdev]

Why need this change? / Root cause:

• Fixed no password when create public room

Changes made:

Test Scope / Change impact:

Issue

• Fix: 新創公開房間不帶 password #263

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
game-lobby-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 14, 2023 5:33am

[bxbdev]

@Yuwen-ctw @ttpss930141011
我的 yarn 目前是使用最新版本 3.6.1，不知道怎麼解決 CI 的問題，目前 git push 後，都會一直卡在 CI workflow / Build And LHCI

[Yuwen-ctw]

Hello, bxb
根據錯誤訊息 error Your lockfile needs to be updated, but yarn was run with `--frozen-lockfile`. ，可以嘗試 yarn install 試試，參考 https://gist.github.com/kenmori/2b31d0cde603751cf92afe2e2bb20263。

另外，在與他人共同維護專案時，可能的話均建議採用一致且版本相同的套件管理工具，以減少潛在的風險與衝突；在本專案建構之初，並未針對套件管理工具的版本進行討論，因此預設可能為 yarn@1.22.19 ，同時我也會假設在建構產品時，Docker 也是以該版本進行建構的 ( 目前手邊無電腦暫無法進行確認 )。因此，在維護本專案時，建議可以考慮透過 yarn set version <version> ，進行 yarn 版本的切換。

若經思考後決定維持原版本 (yarn@1.22.19) 進行維護，則建議移除此 PR 中因升級 yarn 而產生的相關檔案或程式碼。

以上說明，有疑慮可以再討論，感謝！

[ttpss930141011]

Hello @bxbdev
關於 CI workflow / Build And LHCI failed 的問題可以看之前的討論：here and here
綜合起來覺得比較有可能的原因有兩項：

1. server 未實作 socket 部分導致 ci 產生未預期的錯誤。
2. Runner source 不夠導致 ci 失敗 (由本地 lighthouse 成功推測來的猜想)
   目前還沒想到要怎麼解，等未來 socket 與 server 串接上之後再來收斂。

[Yuwen-ctw]

Hello @bxbdev ，
程式碼實作部分沒有問題，再麻煩針對現行的衝突進行處理了。

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[GaaSBot]

✅ Knip Scan Result for 933b1bb

Unused files (10)

Unused files (10)
.bundlewatch.config.js
components/shared/Chat/ChatContent.tsx
components/shared/Chat/ChatMessage.tsx
components/shared/Chat/index.tsx
components/shared/Icon/group/news.tsx
configs/i18nConfigs.ts
lighthouserc.js
reset.d.ts
scripts/knipScanReporter.js
scripts/lhciScanReporter.js

Unused dependencies (2)

Unused dependencies (2)
@svgr/webpack package.json
sharp package.json

Unused devDependencies (5)

Unused devDependencies (5)
@actions/github package.json
@octokit/core package.json
@storybook/blocks package.json
@storybook/testing-library package.json
@total-typescript/ts-reset package.json

Configuration issues (1)

Configuration issues (1)
Unused item in ignoreDependencies: @next/bundle-analyzer

[GaaSBot]

🤖 Lighthouse Scan Result for 933b1bb

/rooms

Metric	Value
Performance	64
Seo	80
Accessibility	79
HTML Report for LHCI Scan	Report Link

/rooms/abc

Metric	Value
Performance	74
Seo	80
Accessibility	69
HTML Report for LHCI Scan	Report Link

/login

Metric	Value
Performance	50
Seo	80
Accessibility	97
HTML Report for LHCI Scan	Report Link

[bxbdev]

@Yuwen-ctw CI 確認過沒有問題了！可以進行合併了。謝謝。