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

Owen/refactor #10

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Owen/refactor #10

wants to merge 4 commits into from

Conversation

OwenLu0125
Copy link
Collaborator

@OwenLu0125 OwenLu0125 commented Mar 13, 2024

  • 專案剛開始啟動,會因為 opensea api 回應失敗,造成專案無法啟動
  • 我先把不需要的 opensea api 請求拿掉
  • 後來問題變成有 hydration 問題
  • 透過 isMounted 方式解決
  • 目前發現 mainfold 那邊的圖片空間似乎有問題,如下:
截圖 2024-03-12 下午4 22 27

重構規劃

  • 先在 hooks 的資料夾開一個新的資料夾叫 api
  • 把要使用的 api 都放在 hooks 裡面的 api 下面
  • 把code 更新到 tbw 的 repo

@OwenLu0125
Copy link
Collaborator Author

OwenLu0125 commented Mar 13, 2024

3/13 討論

  • 可以把測試帳號更新在 read.md
  • 把今天的修正,搬到 tbw
  • 更新 opensea api 的問題

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.

1 participant