-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Umbrella Issue][Feature request] More Factory
to Ohshown
renaming in the django damin console
#14
Comments
Hi @tai271828 , could you please check if this issue is still in current main-tbbca branch? |
Renaming suggestion of fields in Ohshown Events Cet review status -->Review Status
Cet report status -->Species check
Building status --> Report type
Usage status --> Attributes
Details
Other fields we will need in the future:
|
Some of the rewording tasks are completed and some of them do not. Next step: |
Factory
to Ohshown
renaming in the django damin consoleFactory
to Ohshown
renaming in the django damin console
@tai271828 @diaoga Could you please explain how did you decide the "Renaming suggestion of fields in Ohshown Events"? Are there any existing relationship or usage in the admin/FE UI flow between the fields and the OhShown Event? |
我不太懂這個問題~~ 這個 issue 單純是把既有的 admin console 中的名字換成比較適合 ohshown 專案的名字;是完全一對一的關係,沒有建立新的 field (不確定我有沒有回答到你 Orz ) |
@tai271828 我想問的比較像是,為什麼是把A改成B,而不是把C改成B? 有點像是 @Neilxx 在3/9 meeting note裡的問題,我們之所以決定這樣改,是有參考過Disfacoty的flow,覺得這樣改比較適合嗎?還是單純就是隨便找個field來rename? |
懂了! quick answer: Yes,參考過才改的,不是隨便改的。每一個 field 都是 @diaoga 按照他認為合理[1]的方式重新命名的。 long answer: 我直接更新 3/9 meeting note 上對應的問題,補上我知道的資訊,感謝 heads-up! @diaoga 再麻煩你看看我說得跟你想得有沒有一樣,如果有不一樣歡迎糾正我喔~~ [1] 合理:根據 @diaoga 的專業加上第一版的 flow chart 與既有的程式碼限制互相配合的結果
|
和 @tai271828 說的一樣,我只是按照我心中對資料架構的了解,用一組一組的方式,把舊的field重新命名。 |
According to 220313 PyCon APAC development sprint note
Is the Renaming of the django admin console will still be an issue? Or we could establish a whole new table? |
@diaoga from what I understand according the the 0313 note, we will establish a whole new table (and kept the original one alive until we refactored it). |
Subtasks
Check list based on #14 (comment)
Cet review status -->Review Status
Cet report status -->Species check
Building status --> Report type
Usage status --> Attributes
Details
Other fields we will need in the future: Track with #34
More Notes of Screenshot That we Need to Rename
The text was updated successfully, but these errors were encountered: