sitcon-tw / Camp2024-Drawing-Competition-Backend

A Python Turtle Judge Challenge System
Other
1 stars 2 forks source link

Bug fixes for Round model and CORS headers #9

Closed burnedinthesky closed 6 months ago

burnedinthesky commented 6 months ago
  1. Round model 目前只會紀錄 Time,(雖然我們不會有這個需求)但如果挑戰進行超過24h就會出bug,所以改成DateTime
  2. CORS少加一個middleware,已補上

描述 Description

這支 PR 所做的事情,盡可能詳細取代本文字。 Replace this text and describe This PR doing(Better to be detail)

相關的 Issues #(ticket number) / 無 Related Issues #(ticket number) / None

變更的類型 Change Type

影響的 API Affect API

相關的 資源或參考文件連結 Related Resource/Document Link

If this PR related some resource/document please offer link

檢查清單 Check List

- [ ] 我已經在本地手動測試過,確保功能的完整性和穩定性 I have done fully correct manual test

fan9704 commented 6 months ago

@burnedinthesky 我當初也是使用 Datetime 後來改成 Time 的用意是方便建立 Round 如果 Round 綁定日期,可能每天的 Round 都要重新開,雖然嚴格來講要用 DateTimeField 但考量我們需要 Test(試玩) 與 Production(正式學員挑戰) 使用同個 Round 每次都開新 Round 其實偏麻煩,可以再考慮看看要不要還是使用 TimeField

burnedinthesky commented 6 months ago

如果是擔心開發不方便把end_time設在很遠以後或開發時直接用Django Admin改end_time應該就好了?

fan9704 commented 6 months ago

Ok 那先這樣,有需要改再說

fan9704 commented 6 months ago

剛剛發現一件事情 發 PR 的 Commit 不要 Master > Master 阿 記得開 Branch

burnedinthesky commented 6 months ago

啊啊對誒 好我之後會記得 感謝