---
# System prepended metadata

title: 9/6 codereview

---

9/6 codereview
--
~~(1) Readme.md的連結刪除 馬~~
---

~~(2) package.json
     dev-mac
     dev-win 擇一
     mac跑 win的指令似乎不會壞~~
     

----

~~(3) pytz 刪除 （k)~~

---


(4)form.py 在名稱form之前可以加上 apps名字
例如 company_form.py (當作參考) (東橙)

----

(5)django-htmx 可刪(k)

---

(6)make file 馬

----

(7)settings的mailgun環境變數 馬

---

(8)django-environ 可刪 馬

----

~~(9) 確認templates大小寫的問題~~


---

(10)Company models

公司員工數：employees
owner_rel：可考慮刪除
平均數 average number 可暫時拿掉，因為還用不到 （東橙）

---

(11)db_table 決定是否要統一客製化，或者不另外給名字

---

~~(12)job 跟 company models當中
欄位當中有describe 跟 description 
需要統一 （東橙）~~


---
(13)job models
skill 技能 manytomany
內容

----
(14)
軟刪除統一命名


-----
(15)

更改forms/form ->forms/users.form
移除`def index(request):`多餘判斷
`users/views.py`新增`@require_POST`
`views.py sign_out`改成`@require_POST` （東橙）

==只改user這個 app==

----

(15)
 if request.method =="POST"
可以改成
@require_POST

除了 user以外的 apps

---

(16)
resume models 會有 n+1 的問題

------

(17)
user的model type_choice 可以改成 roles_choice

並在views當中，判別roles_choice 轉址，可以增加註解
roles_choice == 1 #代表是個人用戶   (k)

-----



(18)

移除`def index(request):`多餘判斷

---


(19)
~~使用者提示訊息~~


----
(20)
~~line_save_profile 改名稱 social_save_profile (丁)~~


----
(21)
~~userinfo updated_at 設 auto_now=True (丁)
移除 TimeStampedModel ~~

