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

Add username ID #50

Merged
merged 6 commits into from
Sep 23, 2024
Merged

Add username ID #50

merged 6 commits into from
Sep 23, 2024

Conversation

k22036
Copy link
Collaborator

@k22036 k22036 commented Sep 23, 2024

確認事項

  • マージ先,マージ元のブランチは正しいですか?

  • mainに直接マージしようとしていませんか?

  • コミット,ファイルで変更点を確認しましたか?

  • Privateな情報が入っていないですか?

実装内容(変更点)

具体的な変更点や修正箇所を箇条書きでリストアップしてください。

  • リファクタリング
    • 無駄に分割されているファイルの統合
  • すれ違いで自身の子供を認識するように
  • 名前にidを付与するように(破壊的変更)
    • これにより同じ名前のユーザーの区別が可能
  • ユーザー一覧取得API

伝達事項

  • 名前のidは破壊的変更です
    • 最新のサーバー側のコードとios側のコードでないと正しく動きません
    • データベースの初期化が必要です
    • 破壊的変更点
      • データベースのユーザーの構造の変更

@k22036 k22036 changed the title Add username Add username ID Sep 23, 2024
@hyouhyan hyouhyan merged commit 09cf48a into dev Sep 23, 2024
1 check passed
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.

2 participants