Normal-OJ / new-front-end

v2 client for Normal Online Judge
https://noj.tw
Mozilla Public License 2.0
0 stars 7 forks source link

Problem Creation/Edition/Deletion Pages #40

Closed uier closed 1 year ago

cloudflare-pages[bot] commented 1 year ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 81f99de
Status: ✅  Deploy successful!
Preview URL: https://d9f0d1cd.nfe.pages.dev
Branch Preview URL: https://feat-forms.nfe.pages.dev

View logs

uier commented 1 year ago

ProblemCard 有需要寫成一個從外部帶錯誤訊息的元件嗎

我不清楚這句話的問題是「有需要從外部帶錯誤訊息」還是「有需要寫成一個元件」,所以我都回答

根據我的經驗這是一個蠻適合使用 container presentational pattern 的場景,並且 ProblemCard 是有被復用於其他地方的,所以寫成一個元件

需不需要從外部帶錯誤訊息,我覺得是不一定的,這也是為什麼我開了 #34 這個 Issue,如果你的問題是這個的話,那我們可以先忽略這隻 PR 上的這個問題,然後去 #34 討論

laporchen commented 1 year ago

對 就是 #34 那個 所以現在還沒處理那支 issue 前就一樣先用這個 ProblemCard 就好了對吧

uier commented 1 year ago

對 就是 #34 那個 所以現在還沒處理那支 issue 前就一樣先用這個 ProblemCard 就好了對吧

Yes