mirror of
https://github.com/siop-spelev/siop2.git
synced 2026-08-08 12:41:54 +00:00
fix(r6): recette Gestionnaire — Menu non scrollable, BC sans confirmation
Deux bugs réels trouvés sur iPhone physique par le référent : 1. app/(tabs)/menu.tsx n'était pas scrollable (View, pas ScrollView) — avec les 4 groupes désormais pleinement câblés (12 liens + 4 en-têtes), le contenu dépasse la hauteur de l'écran et le dernier groupe (Pilotage, dont Personnes & équipes) était strictement inaccessible. Présent depuis R6.1, seulement révélé maintenant. Fix : ScrollView autour de la liste des groupes, en-tête fixe au-dessus. 2. app/stock/[id]/commander.tsx naviguait silencieusement en arrière après succès — le référent a créé deux BC (vérifiés corrects côté serveur) sans aucun retour dans l'app. Fix : écran de confirmation explicite (référence, fournisseur, total, statut) avant de revenir. Amélioration sur retour direct : Tiers gagne une fiche détail (app/tiers/[id].tsx — identité, contact, BC en cours, sites rattachés, aucune nouvelle requête) — le lecture seule sans réaction au tap se lisait comme un écran cassé plutôt que délibérément simple. Typecheck propre, 17 tests Jest, lint 5/5 paquets. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -1,5 +1,5 @@
|
||||
import { router } from 'expo-router';
|
||||
import { Pressable, Text, View } from 'react-native';
|
||||
import { Pressable, ScrollView, Text, View } from 'react-native';
|
||||
import { SafeAreaView } from 'react-native-safe-area-context';
|
||||
import type { ObjectCategory, PermissionRight } from '@siop/shared';
|
||||
import { useMe } from '@/auth/session';
|
||||
@@ -75,32 +75,34 @@ export default function PageMenu() {
|
||||
|
||||
return (
|
||||
<SafeAreaView style={{ flex: 1, backgroundColor: t.fond }} edges={['top']}>
|
||||
<View style={{ padding: 14, gap: 14, flex: 1 }}>
|
||||
<View style={{ padding: 14, gap: 10, flex: 1 }}>
|
||||
<EnteteTabs />
|
||||
<Text style={{ fontFamily: 'Manrope_800ExtraBold', fontSize: 19, color: t.encre }}>Menu</Text>
|
||||
{groupesVisibles.length === 0 ? (
|
||||
<Text style={{ color: t.encre3, fontFamily: 'Manrope_600SemiBold', textAlign: 'center', padding: 24 }}>
|
||||
Rien d'autre accessible à votre rôle ici.
|
||||
</Text>
|
||||
) : null}
|
||||
{groupesVisibles.map((g) => (
|
||||
<View key={g.titre} style={{ gap: 6 }}>
|
||||
<Text
|
||||
style={{
|
||||
fontFamily: 'Manrope_800ExtraBold',
|
||||
fontSize: 10.5,
|
||||
letterSpacing: 0.8,
|
||||
textTransform: 'uppercase',
|
||||
color: t.encre3,
|
||||
}}
|
||||
>
|
||||
{g.titre}
|
||||
<ScrollView contentContainerStyle={{ gap: 14, paddingBottom: 16 }} showsVerticalScrollIndicator={false}>
|
||||
{groupesVisibles.length === 0 ? (
|
||||
<Text style={{ color: t.encre3, fontFamily: 'Manrope_600SemiBold', textAlign: 'center', padding: 24 }}>
|
||||
Rien d'autre accessible à votre rôle ici.
|
||||
</Text>
|
||||
{g.liens.map((lien) => (
|
||||
<LigneMenu key={lien.libelle} lien={lien} t={t} />
|
||||
))}
|
||||
</View>
|
||||
))}
|
||||
) : null}
|
||||
{groupesVisibles.map((g) => (
|
||||
<View key={g.titre} style={{ gap: 6 }}>
|
||||
<Text
|
||||
style={{
|
||||
fontFamily: 'Manrope_800ExtraBold',
|
||||
fontSize: 10.5,
|
||||
letterSpacing: 0.8,
|
||||
textTransform: 'uppercase',
|
||||
color: t.encre3,
|
||||
}}
|
||||
>
|
||||
{g.titre}
|
||||
</Text>
|
||||
{g.liens.map((lien) => (
|
||||
<LigneMenu key={lien.libelle} lien={lien} t={t} />
|
||||
))}
|
||||
</View>
|
||||
))}
|
||||
</ScrollView>
|
||||
</View>
|
||||
</SafeAreaView>
|
||||
);
|
||||
|
||||
@@ -2,13 +2,19 @@ import { router, useLocalSearchParams } from 'expo-router';
|
||||
import { useState } from 'react';
|
||||
import { ScrollView, Text, TextInput, View } from 'react-native';
|
||||
import { SafeAreaView } from 'react-native-safe-area-context';
|
||||
import type { PurchaseOrderDto } from '@siop/shared';
|
||||
import { useCreatePurchaseOrder, usePart } from '@/api/ressources';
|
||||
import { BoutonTel, Carte, EnteteFiche, LigneInfo } from '@/composants/ui';
|
||||
import { useTokens } from '@/theme/tokens';
|
||||
|
||||
const fmtMAD = new Intl.NumberFormat('fr-FR', { style: 'currency', currency: 'MAD' });
|
||||
|
||||
/** Nouveau BC pré-rempli depuis l'alerte stock — maquette écran 6. Une
|
||||
* seule ligne (cette pièce) : le bon de commande multi-lignes détaillé
|
||||
* reste au web (D4). */
|
||||
* reste au web (D4). Confirmation explicite après création (trouvé en
|
||||
* recette 02/08 : la navigation instantanée en cas de succès ne donnait
|
||||
* aucun retour visible — deux BC créés côté serveur sans que le référent
|
||||
* s'en aperçoive dans l'app). */
|
||||
export default function PageCommander() {
|
||||
const t = useTokens();
|
||||
const { id } = useLocalSearchParams<{ id: string }>();
|
||||
@@ -17,8 +23,33 @@ export default function PageCommander() {
|
||||
const manquant = piece ? Math.max(piece.threshold - piece.stock, 1) : 1;
|
||||
const [quantite, setQuantite] = useState(String(manquant));
|
||||
const [prix, setPrix] = useState('');
|
||||
const [cree, setCree] = useState<PurchaseOrderDto | null>(null);
|
||||
|
||||
if (!piece || !piece.supplierId) return null;
|
||||
|
||||
if (cree) {
|
||||
return (
|
||||
<SafeAreaView style={{ flex: 1, backgroundColor: t.fond }} edges={['top']}>
|
||||
<ScrollView contentContainerStyle={{ padding: 14, gap: 12 }}>
|
||||
<EnteteFiche titre="BC créé" />
|
||||
<Carte titre="✓ Bon de commande enregistré">
|
||||
<LigneInfo nom="Référence" valeur={cree.reference} />
|
||||
<LigneInfo nom="Fournisseur" valeur={cree.supplierName} />
|
||||
<LigneInfo nom="Total" valeur={fmtMAD.format(cree.total)} />
|
||||
<LigneInfo nom="Statut" valeur="Brouillon" />
|
||||
</Carte>
|
||||
<Text style={{ color: t.encre3, fontFamily: 'Manrope_400Regular', fontSize: 11.5, textAlign: 'center' }}>
|
||||
L'envoi au fournisseur se fait depuis le web pour l'instant.
|
||||
</Text>
|
||||
<BoutonTel
|
||||
libelle="Terminé"
|
||||
surAppui={() => (router.canGoBack() ? router.back() : router.replace('/stock'))}
|
||||
/>
|
||||
</ScrollView>
|
||||
</SafeAreaView>
|
||||
);
|
||||
}
|
||||
|
||||
const prixDefaut = piece.lastUnitPrice != null ? String(piece.lastUnitPrice) : '';
|
||||
const prixSaisi = prix || prixDefaut;
|
||||
const qte = Number(quantite);
|
||||
@@ -89,7 +120,7 @@ export default function PageCommander() {
|
||||
supplierId: piece.supplierId!,
|
||||
lines: [{ partId: piece.id, quantity: qte, unitPrice: pu }],
|
||||
},
|
||||
{ onSuccess: () => (router.canGoBack() ? router.back() : router.replace('/stock')) },
|
||||
{ onSuccess: (bc) => setCree(bc) },
|
||||
)
|
||||
}
|
||||
/>
|
||||
|
||||
48
apps/mobile/app/tiers/[id].tsx
Normal file
48
apps/mobile/app/tiers/[id].tsx
Normal file
@@ -0,0 +1,48 @@
|
||||
import { useLocalSearchParams } from 'expo-router';
|
||||
import { ScrollView, Text } from 'react-native';
|
||||
import { SafeAreaView } from 'react-native-safe-area-context';
|
||||
import { usePartners } from '@/api/ressources';
|
||||
import { Carte, EnteteFiche, LigneInfo } from '@/composants/ui';
|
||||
import { useTokens } from '@/theme/tokens';
|
||||
|
||||
/** Fiche tiers — R6.3, ajoutée en recette (02/08). Consultation seule (D4) :
|
||||
* identité, contact, activité (BC en cours pour un fournisseur, sites
|
||||
* rattachés pour un client/syndic) — déjà connu de la liste, pas de
|
||||
* requête supplémentaire. Création/édition restent au web. */
|
||||
export default function PageFicheTiers() {
|
||||
const t = useTokens();
|
||||
const { id } = useLocalSearchParams<{ id: string }>();
|
||||
const { data: partners } = usePartners();
|
||||
const p = (partners ?? []).find((x) => x.id === id);
|
||||
if (!p) return null;
|
||||
|
||||
return (
|
||||
<SafeAreaView style={{ flex: 1, backgroundColor: t.fond }} edges={['top']}>
|
||||
<ScrollView contentContainerStyle={{ padding: 14, gap: 10 }}>
|
||||
<EnteteFiche titre={p.name} />
|
||||
<Carte titre="Identité">
|
||||
<LigneInfo nom="Type" valeur={p.kind === 'SUPPLIER' ? 'Fournisseur' : 'Client'} />
|
||||
<LigneInfo nom="Statut" valeur={p.isActive ? 'Actif' : 'Désactivé'} />
|
||||
<LigneInfo nom="Contact" valeur={p.contactName ?? '—'} />
|
||||
<LigneInfo nom="Téléphone" valeur={p.phone ?? '—'} />
|
||||
<LigneInfo nom="E-mail" valeur={p.email ?? '—'} />
|
||||
<LigneInfo nom="Ville" valeur={p.city ?? '—'} />
|
||||
</Carte>
|
||||
{p.kind === 'SUPPLIER' ? (
|
||||
<Carte titre="Achats">
|
||||
<LigneInfo nom="BC en cours" valeur={String(p.openOrders)} />
|
||||
</Carte>
|
||||
) : null}
|
||||
{p.siteNames.length ? (
|
||||
<Carte titre="Sites rattachés">
|
||||
{p.siteNames.map((s) => (
|
||||
<Text key={s} style={{ fontFamily: 'Manrope_600SemiBold', fontSize: 12.5, color: t.encre }}>
|
||||
{s}
|
||||
</Text>
|
||||
))}
|
||||
</Carte>
|
||||
) : null}
|
||||
</ScrollView>
|
||||
</SafeAreaView>
|
||||
);
|
||||
}
|
||||
@@ -1,11 +1,14 @@
|
||||
import { FlatList, Text, View } from 'react-native';
|
||||
import { router } from 'expo-router';
|
||||
import { FlatList, Pressable, Text, View } from 'react-native';
|
||||
import { SafeAreaView } from 'react-native-safe-area-context';
|
||||
import { usePartners } from '@/api/ressources';
|
||||
import { EnteteFiche } from '@/composants/ui';
|
||||
import { useTokens } from '@/theme/tokens';
|
||||
|
||||
/** Tiers — R6.3 (Ressources). Consultation seule sur mobile (D4) : la
|
||||
* création/édition de fournisseurs et clients reste au web pour l'instant. */
|
||||
* création/édition de fournisseurs et clients reste au web pour l'instant.
|
||||
* Fiche détail ajoutée en recette (02/08) : le tap sans réaction se lisait
|
||||
* comme un écran cassé plutôt que délibérément simple. */
|
||||
export default function PageTiers() {
|
||||
const t = useTokens();
|
||||
const { data: partners } = usePartners();
|
||||
@@ -25,7 +28,9 @@ export default function PageTiers() {
|
||||
</Text>
|
||||
}
|
||||
renderItem={({ item: p }) => (
|
||||
<View
|
||||
<Pressable
|
||||
accessibilityRole="button"
|
||||
onPress={() => router.push(`/tiers/${p.id}`)}
|
||||
style={{
|
||||
backgroundColor: t.surface,
|
||||
borderColor: t.bordure,
|
||||
@@ -58,7 +63,7 @@ export default function PageTiers() {
|
||||
{[p.contactName, p.phone, p.city].filter(Boolean).join(' · ') || '—'}
|
||||
{p.kind === 'SUPPLIER' && p.openOrders ? ` · ${p.openOrders} BC en cours` : ''}
|
||||
</Text>
|
||||
</View>
|
||||
</Pressable>
|
||||
)}
|
||||
/>
|
||||
</View>
|
||||
|
||||
Reference in New Issue
Block a user