fix: harden rider profile onboarding and mobile save access
This commit is contained in:
parent
c380f10f21
commit
79f6f3e526
|
|
@ -232,7 +232,7 @@ class _RiderProfileScreenState extends ConsumerState<RiderProfileScreen> {
|
||||||
}
|
}
|
||||||
|
|
||||||
return SingleChildScrollView(
|
return SingleChildScrollView(
|
||||||
padding: const EdgeInsets.all(16.0),
|
padding: const EdgeInsets.fromLTRB(16.0, 16.0, 16.0, 100.0),
|
||||||
child: Form(
|
child: Form(
|
||||||
key: _formKey,
|
key: _formKey,
|
||||||
child: Column(
|
child: Column(
|
||||||
|
|
|
||||||
|
|
@ -150,7 +150,7 @@ class RiderProfileService {
|
||||||
|
|
||||||
final previousHandle = existingProfile?.handle;
|
final previousHandle = existingProfile?.handle;
|
||||||
final isChangingHandle =
|
final isChangingHandle =
|
||||||
previousHandle != null && previousHandle != normalizedHandle;
|
previousHandle != null && previousHandle.isNotEmpty && previousHandle != normalizedHandle;
|
||||||
|
|
||||||
final updatedProfile = RiderProfile(
|
final updatedProfile = RiderProfile(
|
||||||
uid: uid,
|
uid: uid,
|
||||||
|
|
@ -183,7 +183,7 @@ class RiderProfileService {
|
||||||
});
|
});
|
||||||
|
|
||||||
// 3. Release previous handle if changed
|
// 3. Release previous handle if changed
|
||||||
if (isChangingHandle) {
|
if (isChangingHandle && previousHandle.isNotEmpty) {
|
||||||
final oldHandleRef = firestore.collection('handles').doc(previousHandle);
|
final oldHandleRef = firestore.collection('handles').doc(previousHandle);
|
||||||
transaction.delete(oldHandleRef);
|
transaction.delete(oldHandleRef);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue