Skip to content

Commit 893cb3f

Browse files
alexpmicheletclaude
andcommitted
chore(uber-direct): supprime non-null assertion sur uberState.uberCustomerId
Lint error `@typescript-eslint/no-non-null-assertion` sur le `!` que j'avais mis dans le useEffect du pré-remplissage. Réécrit en extrayant le storedCustomerId AVANT le useEffect (null-safe), élimine le besoin du `!` + simplifie la deps array. Pas de changement comportement. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 43d84b0 commit 893cb3f

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

  • apps/admin/src/app/(app)/t/[tenantId]/parametres/uber-direct

apps/admin/src/app/(app)/t/[tenantId]/parametres/uber-direct/page.tsx

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -78,15 +78,13 @@ export default function UberDirectSettingsPage(): React.JSX.Element {
7878
// la connaît (sinon l'opérateur doit le retaper alors qu'on le connaît).
7979
// Les secrets (client_id, client_secret, webhook_signing_key) restent vides
8080
// — jamais ré-exposés côté wire (envelope encrypted server-side).
81+
const storedCustomerId =
82+
uberState !== undefined ? uberState.uberCustomerId : null;
8183
useEffect(() => {
82-
if (
83-
uberState !== undefined &&
84-
uberState.uberCustomerId !== null &&
85-
form.customerId === ""
86-
) {
87-
setForm((prev) => ({ ...prev, customerId: uberState.uberCustomerId! }));
84+
if (storedCustomerId !== null && form.customerId === "") {
85+
setForm((prev) => ({ ...prev, customerId: storedCustomerId }));
8886
}
89-
}, [uberState, form.customerId]);
87+
}, [storedCustomerId, form.customerId]);
9088

9189
if (uberState === undefined) {
9290
return (

0 commit comments

Comments
 (0)