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.

Sign up for GitHub

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

Jump to bottom

[主線] [PTT] 實作使用者設定 -- 修改usecase和repository #148

Closed
kyho4515 opened this issue Mar 23, 2021 · 4 comments · Fixed by #150
Closed

[主線] [PTT] 實作使用者設定 -- 修改usecase和repository #148

kyho4515 opened this issue Mar 23, 2021 · 4 comments · Fixed by #150
Labels
help wanted Extra attention is needed

Comments

@kyho4515
Copy link
Contributor

實作細節 / Details of Implement

  • 修改 internal/usecase/user.go
    • 修改GetUserPreferences內的data
  • 修改 internal/repsitory/user.go
    • 新增dummy function給usecase

期程 / Schedule

  • 討論時間： 兩天, 到 3/25
  • 實作時間： 兩天, 到 3/27
  • 確認時間： 兩天, 到 3/29

相關文件 / Documents

主issue: #62
PTT 後端系統協定
@kyho4515 kyho4515 added the help wanted Extra attention is needed label Mar 23, 2021
@kyho4515
Copy link
Contributor Author

想確認一下repository的寫法, 因為go-bbs還沒有相對應的method, 目前這部分是直接對每個usecase裡的field都建一個dummy function嗎?

@PichuChen
Copy link
Member

想確認一下repository的寫法, 因為go-bbs還沒有相對應的method, 目前這部分是直接對每個usecase裡的field都建一個dummy function嗎?

對

@PichuChen
Copy link
Member

@kyho4515 可以問一下目前這個 ISSUE 的狀態嗎？

@kyho4515
Copy link
Contributor Author

@kyho4515 可以問一下目前這個 ISSUE 的狀態嗎？

這幾天比較忙一點, 這部分已經改好, 準備送出pull request了

@kyho4515 kyho4515 mentioned this issue Mar 29, 2021
Merged
@PichuChen PichuChen mentioned this issue Mar 31, 2021
Closed
3 tasks
PichuChen added a commit that referenced this issue Apr 2, 2021
@PichuChen
Merge pull request #150 from kyho4515/feature/#148-#user_preferences_…
c796368 
…usecase_and_repository

Feature/#148 #user preferences usecase and repository
@PichuChen PichuChen mentioned this issue Jun 20, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/#148 #user preferences usecase and repository kyho4515/Ptt-backend
2 participants
@PichuChen @kyho4515