発表用媒体(PDF, etc)を返すエンドポイントを作成

[Cj-bc]

Step 1: 目的

• 発表用に印刷するファイルを作成し、返す

Step 2: 概要

• 仕様: 一個前のtime_indexの各lotteryのwinnersのpublic_idを、classroomごとにわけ、かつその中でstudent/visitorで分けて印刷可能にしたものを返す。作成した媒体はキャッシュする。

• 既に作成されている場合は、キャッシュされたものを返す

• エンドポイント名は、PDFという文字列を含まないもの

• rfc: /results -- 権限範囲 rfcs#2

タスク:

• 返す形式を決定(PDF or html)

• PDF or HTMLのテンプレートを作成する(おそらくjinja)

  • html部分の作成
  • css部分の作成

• エンドポイント名を決定

• 権限範囲決定 ( /results -- 権限範囲 rfcs#2)

• spec作成

• エンドポイントを作成

  • 1. Get previous time index
  • 2. Get previous lotteries using index
  • 5. Make 2 public_id lists, based on user's 'kind'('student', 'visitor')
  • 6. Send them to the jinja template

• get_prev_time_index追加

• test作成

  • test: test_prev_time_index_ooa
  • test: test_prev_time_index_lim
  • test: test_prev_time_index_same

決定事項:

• 形式: PDF HTML(システムの都合上PDF化は時間がかかりそうでした)
• PDFを出力する: pdfkit
• エンドポイント名: /render_results
• HTTP method: GET
• 権限制約: なし
• 時間外だった場合のエラーコードおよびstatus code
  • error_code: 06
  • HTTP status code: 400
  • error_message: not acceptable time

[Cj-bc]

FlaskでPDFを返す方法: https://aroundthedistance.hatenadiary.jp/entry/2015/01/18/234425

[Cj-bc]

HTTP method: GET
path: /results

[Cj-bc]

rfc has been opened Sakuten/rfcs#2

[Cj-bc]

権限範囲は決定しないことになりました。

[Cj-bc]

時間外だった場合のエラーコードおよびstatus code
error_code: 06
HTTP status code: 404
error_message: not acceptable time

[Cj-bc]

get_prev_time_index仕様

• あるindexの応募受付時間終了時+1分〜次回の応募受付時間終了時+1分までの間、そのindexを返します

[coord-e]

時間外、400では？

[Cj-bc]

あ、ほんとだww
単純なミスですありがと

[Cj-bc]

template作成について

• 横に並べられるIDの数はいくつか
• kindで分けて発表とあるが、紙を分けて発表するという認識で合っているか

[Cj-bc]

データ構造的によろしくないかもしれませんが、ちょっと進ませてくだせぇ..

[Cj-bc]

配列の分け方、リファクタリングが必要な部分です。

data = {'kinds':                                                         
          [{'visitor':                                                   
              [{'classroom_id': lottery.classroom_id,                    
                'winners': [lottery.application.user(has 'visitor' kind)]
               }],                                                       
            'student':                                                   
              [{'classroom_id': lottery.classroom_id,                    
                'winners': [lottery.application.user(has 'student' kind)]
               }]                                                        
           }]                                                            
       }                                                                 

[Cj-bc]

PDFきつそうなので一度HTMLで返します
PDFきつい理由: wkhtmltopdfが、alpine上でcss適用がうまくできないという情報を取得したから
参考:
https://gist.github.com/akihiromukae/288b163d538d45a197b3f1b54ef385e8

PDFで返すissueも立てておきますが、first_release後になる可能性があります

[Cj-bc]

又、HTMLに変更したことにより、cacheを持つ必要も無くなったので、cache関連のtodoを削除します

[Cj-bc]

🤔
あとはcss書けば終わりです

[shino16]

A0サイズををCSSで表現するとき
仕様では841mmですが、それだとはみ出ます
適正サイズはフォントサイズによって変わるようです
小さめに調整します

[Cj-bc]

close with merging of #212