Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

受付に利用する参加表のプロフィールとタイムテーブルの実装 #2296

Merged
merged 12 commits into from
May 19, 2024

Conversation

oshiro3
Copy link
Contributor

@oshiro3 oshiro3 commented May 1, 2024

QRコードの提示を含む受付参加表についての基礎を実装しました。

こちらを事前印刷もしくは可能であれば当日受付で印刷するなどして利用できればと思っています。
左上セルについては4つ折りで一番上に来る部分(フロントセル)で、受付用QRコードと簡単なプロフィールとして参加種別や公開プロフィールが登録されていればアイコン・名前・TwitterID 程度を記載します。
公開プロフィールが無い場合はQRコードのみ表示します。

右上は個人の登録セッションごとのタイムテーブルを表示します。

下半分については

下のContent3, 4はぶち抜きで会場MAPを表示するのが良いかなと思います

で行こうと思いますが、次P/Rでの実装にできれば。

後続タスクとしては

  • 下半分の会場地図の実装
  • PDF 保存の実装
  • (上2つが完了すれば)ルーティングのページ実装

で考えています。

@oshiro3 oshiro3 requested review from jacopen and takaishi May 1, 2024 14:14
@oshiro3 oshiro3 self-assigned this May 1, 2024
Copy link

github-actions bot commented May 1, 2024

@github-actions github-actions bot added the reviewapps Build ReviewApp environment automatically if this label is granted label May 1, 2024
@jacopen
Copy link
Collaborator

jacopen commented May 1, 2024

原案ありがとうございます!
右上は登録セッション表示に賛成です。
下のContent3, 4はぶち抜きで会場MAPを表示するのが良いかなと思います

@jacopen
Copy link
Collaborator

jacopen commented May 1, 2024

ひとつ気になるのが、ブラウザ表示の印刷だと、この画面のURLやタイトルが勝手に追加されて印刷されるなどで意図した通りの印刷にならないんじゃないかという点が気になってます。
それを防ぐにはPDFで出さないといけないかもなぁと

@ShotaKitazawa ShotaKitazawa removed the reviewapps Build ReviewApp environment automatically if this label is granted label May 5, 2024
Copy link

@github-actions github-actions bot added the reviewapps Build ReviewApp environment automatically if this label is granted label May 11, 2024
@oshiro3 oshiro3 marked this pull request as ready for review May 11, 2024 01:37
@oshiro3 oshiro3 changed the title [WIP] 受付に利用する参加表の原案の実装 受付に利用する参加表のプロフィールとタイムテーブルの実装 May 11, 2024
@oshiro3
Copy link
Contributor Author

oshiro3 commented May 11, 2024

デザイン関係はこちらで
https://cloudnativedays.slack.com/archives/C015CN1PQ06/p1715391116042849

app/javascript/packs/qr_reader1.js Outdated Show resolved Hide resolved
@@ -0,0 +1,16 @@
<!DOCTYPE html>
Copy link
Contributor

Choose a reason for hiding this comment

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

このページへの導線がないですが、直アクセスのみの想定でしょうか?どこかに導線を用意しておいてもいいかなと思いました

Copy link
Contributor Author

Choose a reason for hiding this comment

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

確認ありがとうございます。
まだ下半分ができていないのでそちらを実装してから導線を付けようかと思っています!

Copy link
Contributor

Choose a reason for hiding this comment

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

印刷リンクをつけてもらったと思いますが、一通り完成して、参加者に案内するタイミングで表示する方が望ましいのではと思い直しました。この後参加票のコンテンツが変わることはありますし、ギリギリまでパブリックにはしない方が混乱を避けられそうです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

承知です!
この導線のコミットはリバートしました!

@oshiro3 oshiro3 requested a review from takaishi May 13, 2024 23:22
@oshiro3 oshiro3 marked this pull request as draft May 14, 2024 14:21
@oshiro3
Copy link
Contributor Author

oshiro3 commented May 14, 2024

すいません、残りの実装(導線作成、地図表示、PDFダウンロード)もそんなに重たくなさそうなので一気にこの P/R で入れてしまおうと思います。。。

@oshiro3 oshiro3 marked this pull request as ready for review May 15, 2024 14:30
@oshiro3
Copy link
Contributor Author

oshiro3 commented May 15, 2024

@jacopen 参加表下部の地図表示についてはまだフロアマップできてないかと思うので floor_map.png にロゴ画像をコピーしているので完成したら差し替えてもらうというのが良いかなと思っています!

Copy link

Simplecov Report

Covered Threshold
61.09% 60%

Copy link
Contributor

@takaishi takaishi left a comment

Choose a reason for hiding this comment

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

LGTM!

@oshiro3 oshiro3 merged commit 856a75e into main May 19, 2024
5 checks passed
@oshiro3 oshiro3 deleted the receptions branch May 19, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewapps Build ReviewApp environment automatically if this label is granted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants