Skip to content

Comments

Migration and model#1

Merged
hao0516hsu merged 3 commits intomasterfrom
migration
Aug 22, 2023
Merged

Migration and model#1
hao0516hsu merged 3 commits intomasterfrom
migration

Conversation

@Zephzer
Copy link
Owner

@Zephzer Zephzer commented Aug 22, 2023

migration 跟 model 的測試都通過
最後 外鍵的部分還是用 user_id 用 AC給的測試會過不了
Model 的 XXX.init 的部分 是用 UserId、 TweetId 、 followerId、 followingId 要注意一下

@hao0516hsu
Copy link
Collaborator

  1. User的cover需要加上defaultValue,來源是UI的山景圖
  2. User的avatar需要加上defaultValue,來源是UI的灰底ac圖

Copy link
Collaborator

@hao0516hsu hao0516hsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看起來沒問題了!

@hao0516hsu hao0516hsu merged commit 08bc56d into master Aug 22, 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

Successfully merging this pull request may close these issues.

4 participants