Skip to content

Conversation

genj11
Copy link
Contributor

@genj11 genj11 commented Jul 20, 2024

ネームカード作成・シェアページの基本的な実装を行いました。

  • ネームカード情報の取得・登録機能の実装
  • シェアカードページの実装

プレビュー環境での確認対応がまだできておらず、お手数ですがお手元で確認していただきたいです。
https://localhost:3000/namecard/

シェアカードページ
https://localhost:3000/namecard/be9367b9-4612-4b17-834b-f303cc96dc45/share/

未実装

  • モバイル版スタイル
  • エラー・バリデーションハンドリング
  • SNSシェア機能
  • 完了画面からシェアページへのリンク設定等

@genj11 genj11 requested a review from jiyuujin July 20, 2024 21:09
@genj11 genj11 self-assigned this Jul 20, 2024
Copy link

netlify bot commented Jul 20, 2024

Deploy Preview for vuefes-2024 ready!

Name Link
🔨 Latest commit b4f7eba
🔍 Latest deploy log https://app.netlify.com/sites/vuefes-2024/deploys/669c94b6d49f2c0008159349
😎 Deploy Preview https://deploy-preview-212--vuefes-2024.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

Deploying vuefes-2024-designsystem with  Cloudflare Pages  Cloudflare Pages

Latest commit: b4f7eba
Status: ✅  Deploy successful!
Preview URL: https://82108fee.vuefes-2024-designsystem.pages.dev
Branch Preview URL: https://feature-creation-namecard.vuefes-2024-designsystem.pages.dev

View logs

</script>

<template>
<!-- TODO 画像アップローダーのブラッシュアップ -->
<VFDragDropArea
<label
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

既存のものは使えなかったんですかね

e.preventDefault()
navigateTo(`/namecard/${route.params.id}/edit/complete/`)
}
const receiptId = ref('')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS

昨年もそうだったかと記憶していますが、
編集時は receiptId 編集不可 (いわゆる disabled 状態) となるかもしれないです mm

@jiyuujin
Copy link
Collaborator

jiyuujin commented Jul 21, 2024

memo

/namecard より Google ログインを選択して実行すると、
追加画面のリンクへ飛ばず以下のリンクへ飛ばされ、以後それらが表示されなさそうでした mm

https://localhost:3000/namecard/null/?code=ba7e2a31-3f7c-4669-bd3d-a16274bf6513

=====

role が NULL のまま?だからですかね

@vuejs-jp/web:dev:  ERROR  [nuxt] [request error] [unhandled] [500] Pattern matching error: no pattern matches value ""
@vuejs-jp/web:dev:   at exhaustive (/Users/yuukit/projects/vuefes-2024/node_modules/ts-pattern/src/match.ts:124:11)  
@vuejs-jp/web:dev:   at Proxy.backgroundColor (/Users/yuukit/projects/vuefes-2024/packages/composable/lib/useUserRole.ts:4:244)  
@vuejs-jp/web:dev:   at _sfc_ssrRender (/Users/yuukit/projects/vuefes-2024/packages/ui/components/namecard/NamecardAvatar.vue:38:24)  
@vuejs-jp/web:dev:   at renderComponentSubTree (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:695:9)  
@vuejs-jp/web:dev:   at renderComponentVNode (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:641:12)  
@vuejs-jp/web:dev:   at Module.ssrRenderComponent (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:84:10)  
@vuejs-jp/web:dev:   at _sfc_ssrRender (/Users/yuukit/projects/vuefes-2024/packages/ui/components/namecard/Namecard23.vue:62:31)  
@vuejs-jp/web:dev:   at renderComponentSubTree (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:695:9)  
@vuejs-jp/web:dev:   at renderComponentVNode (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:641:12)  
@vuejs-jp/web:dev:   at Module.ssrRenderComponent (/Users/yuukit/projects/vuefes-2024/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:84:10)

Copy link
Collaborator

@jiyuujin jiyuujin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

一旦マージいただいて OK です

@jiyuujin jiyuujin merged commit d12050d into main Jul 21, 2024
9 of 10 checks passed
@jiyuujin jiyuujin deleted the feature/creation-namecard branch July 21, 2024 09:50
</div>
<CreationStatus :status-key="statusKey" size="small" class="creation-status" />
<!-- TODO 24に置き換え&smallサイズ指定 -->
<VFNamecard23 :user="attendee" class="namecard-preview" />
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✍️

こちらをはじめ VFNamecard23 になっている箇所を VFNamecard24 へ書き換える to @totocalcio

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

画面確認できるようになったら、書き換えて確認してみます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants