Skip to content

feat: implement complete profile and edit profile screens for android#162

Merged
erinc00 merged 2 commits into
developmentfrom
feature/android-profile-flow
Mar 29, 2026
Merged

feat: implement complete profile and edit profile screens for android#162
erinc00 merged 2 commits into
developmentfrom
feature/android-profile-flow

Conversation

@gulcetahtasiz

@gulcetahtasiz gulcetahtasiz commented Mar 28, 2026

Copy link
Copy Markdown
Contributor

Android Profile Flow

Added

  • Complete Profile screen (after signup)
  • Edit Profile screen
  • Profile data layer (ProfileData, ProfileRepository)
  • Reusable components (GenderSelector, LocationSelector, SaveActionBar)

Updates

  • Navigation flow: Signup → Verify → Complete Profile → Profile
  • Edit Profile navigation added

Notes

  • Backend integration not included yet (local state used)
  • Location data is hardcoded for MVP
  • Age will be derived from birthDate on backend side

closes #163

@AlperKartkaya
AlperKartkaya requested a review from erinc00 March 29, 2026 08:12
@erinc00
erinc00 removed the request for review from AlperKartkaya March 29, 2026 08:28

@kckagancan kckagancan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There could be few improvements related to fields in profile and complete profile page, but in general this looks fine. We will do the improvements after MVP

@erinc00 erinc00 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I reviewed the changes and everything looks good to me. The implementation is clear, the overall structure makes sense, and I did not notice any major issues. Nice work on this. Approving the PR.

@erinc00
erinc00 merged commit 49798c7 into development Mar 29, 2026
@kckagancan
kckagancan deleted the feature/android-profile-flow branch March 29, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants