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

バグ修正 #2

Merged
merged 7 commits into from
Jun 21, 2021
Merged

バグ修正 #2

merged 7 commits into from
Jun 21, 2021

Conversation

Kyoya-prog
Copy link
Owner

@Kyoya-prog Kyoya-prog commented Jun 21, 2021

これに記載されている情報をもとに、下記のバグの修正を行なった。

  • レイアウトエラー
    • 各リポジトリ画面のstackViewのy方向の制約が不足していたため追加した
  • メモリリーク
     - クロージャー内でのselfへの参照を弱参照にした
  • パースエラー
    • watchers_countの取り出しの際にrepo["wachers_count"]とtypoになっており、うまく取り出せていなかった
    • 今後の改修も見越して、Codableを使って安全にパースするようにした

@Kyoya-prog Kyoya-prog changed the title バグ修正ブランチ バグ修正 Jun 21, 2021
@Kyoya-prog Kyoya-prog merged commit baae438 into master Jun 21, 2021
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