Skip to content
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

對課程評論 / 留言 #23

Open
1 of 4 tasks
Tracked by #33
uier opened this issue Mar 10, 2023 · 11 comments
Open
1 of 4 tasks
Tracked by #33

對課程評論 / 留言 #23

uier opened this issue Mar 10, 2023 · 11 comments
Assignees

Comments

@uier
Copy link
Contributor

uier commented Mar 10, 2023

Related by #22

@uier uier self-assigned this Mar 10, 2023
@uier
Copy link
Contributor Author

uier commented Apr 8, 2023

@skyhong2002 課程的評論與論壇的貼文是分開的 Entity 嗎?

@skyhong2002
Copy link
Member

抱歉到現在才回,我認為評論與論壇的貼文是不同的 Entity。

評論必須要連結一門課,但論壇貼文不一定
評論除了文字敘述還有 1~5 星數評價,論壇沒有

@JacobLinCool
Copy link
Member

我覺得可以把評價獨立出一個 Entity?評論則與貼文同類型,這樣也可以解決單純評分沒有文字的問題。

erDiagram
  rating }o--|| course : ABOUT
  post }o--o{ rating : LINK
  user ||--o{ rating : RATE_BY
  user ||--o{ post : WRITE_BY
Loading

我不確定會不會有同個評價連結到多個貼文的情境

@Bogay Bogay mentioned this issue Apr 23, 2023
2 tasks
@uier
Copy link
Contributor Author

uier commented May 6, 2023

@Bogay Expected to see the history of comment/rate of an user 這個 task 好像跟 #35 有重疊?我目前還沒有實作,想要怎麼分工?

@Bogay
Copy link
Member

Bogay commented May 6, 2023

我本來打算等你寫好 API 我再抓資料 render

@uier
Copy link
Contributor Author

uier commented May 6, 2023

我本來打算等你寫好 API 我再抓資料 render

Ok 那你期望收到的資料大概是什麼格式?

參數的話應該會提供 offset & count,由近到遠排序

@Bogay
Copy link
Member

Bogay commented May 6, 2023

我預期會有以下資料:

  • 連到特定 comment 的 permalink(不見得要在 API 內直接包含,透過 API 資料組出來也可以?)
  • 課程相關資料:至少有 id,但可以有些名稱之類的重要資料
  • comment 內容,雖然我想是不用全部顯示,但回傳時全部丟應該沒問題?
  • 評分:我不是很確定個人頁面要不要顯示,但感覺也可以做

@uier
Copy link
Contributor Author

uier commented May 6, 2023

我預期會有以下資料:

  • 連到特定 comment 的 permalink(不見得要在 API 內直接包含,透過 API 資料組出來也可以?)
  • 課程相關資料:至少有 id,但可以有些名稱之類的重要資料
  • comment 內容,雖然我想是不用全部顯示,但回傳時全部丟應該沒問題?
  • 評分:我不是很確定個人頁面要不要顯示,但感覺也可以做
  1. comment 目前沒有專屬畫面,所以 link 可能先導到課程資料頁面
  2. 呈上,會給你 course id 以讓你組出課程資料頁面的 link,然後會有課程名稱、老師
  3. ok
  4. ok

@Bogay
Copy link
Member

Bogay commented May 6, 2023

用 anchor 來指向特定 comment 你認為是合理的做法嗎?像 github issue / PR 的 comment 這樣

@uier
Copy link
Contributor Author

uier commented May 6, 2023

用 anchor 來指向特定 comment 你認為是合理的做法嗎?像 github issue / PR 的 comment 這樣

合理吧,但我不知道我有沒有時間做

@Bogay
Copy link
Member

Bogay commented May 6, 2023

也可以是我寫,如果我們有共識的話

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants