Skip to content

fix(discovery): lier la continuation aux critères et au viewer - #522

Open
Gilbert243 wants to merge 529 commits into
mainfrom
fix/discover-a-continuation-context-20261014
Open

Gilbert243 wants to merge 529 commits into
mainfrom
fix/discover-a-continuation-context-20261014

Conversation

@Gilbert243

@Gilbert243 Gilbert243 commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Périmètre A

  • Garde une requête sans filtre valide (comportement existant).
  • Préserve les filtres explicites (comportement existant).
  • Lie maintenant les curseurs signés aux critères de requête et au viewer authentifié, sans accepter de profile_id client.
  • Ne force ni Mayele ni Molongo et n'ajoute ni modèle, ni migration, ni nouveau endpoint.

Compatibilité / limites

  • Les anciens curseurs uniquement basés sur page deviennent invalides ; le client doit recommencer à la première page.
  • Pagination toujours recalculée : le curseur ne garantit pas un snapshot figé.
  • Pas de nouveaux tests ajoutés, conformément à la demande. Exécution CI et validation à effectuer avant merge.
  • Vérification Web exhaustive non réalisée, et changements locaux non visibles via GitHub.

Consolidation du chantier #523 sur cette même branche (2026-10-18)

  • Commerce : prix établis (unknown/free/exact/range), sérialisation à deux décimales et variantes d’Occurrence ; aucun calcul de Quote/Payment déplacé.
  • Opportunities : connaissance publique bornée et réserve explicite sur les faits réels.
  • Services : relation privée Journey→Opportunity et démarrage natif GET/POST /api/v1/services//start/ avec vérification d’autorité et idempotence.
  • Migration additive services.0005_service_start_operation_fingerprint, tests ciblés et intégration CI PostgreSQL.
  • Ancien chantier feat(owners): prix établis, connaissance, relations et démarrage natif Services #523 transféré ici au commit 0ee200b ; ne pas merger séparément.
  • Le périmètre initial décrit ci-dessus reste valable pour Discovery uniquement ; cette consolidation ajoute bien un champ et une migration Services.
  • CI à revérifier sur le nouveau HEAD. Ne pas merger sans CI verte.

Consolidation P2 (#525)

  • PresentationMediaUse, services, selector, migration 0005, tests et contrat doc intégrés dans cette branche ; dépendances Pillow/pypdf corrigées.
  • feat(presentations): P2 media-use ownership and publication contract #525 fermée sans merge après vérification des huit fichiers transférés.
  • Fix ciblés : PermissionDenied sur asset étranger, repr Now ne divulgue pas les identités techniques, retrait du bouton aperçu N1 hors contrat, nettoyage de code Flutter inutilisé.
  • Aucun merge main. Les CI doivent être vertes avant intégration.

CI après consolidation (à résoudre)

  • e2e et django échouent actuellement avant tests, au gate git diff --exit-code -- package-lock.json static/dist : les bundles générés static/dist/makolo.css et static/dist/makolo.js ne correspondent pas à npm run build.
  • Correction requise par le toolchain versionné : npm ci && npm run build, puis commit des deux static/dist générés. Ne pas modifier le gate/test.
  • Flutter CI et shards PostgreSQL toujours en cours lors de cette mise à jour.

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