Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crée une vue sur les restrictions servie par pg_featureserv #1062

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

florimondmanca
Copy link
Collaborator

@florimondmanca florimondmanca commented Nov 7, 2024

Plusieurs petits irritants à traiter :

  • roadType devrait s'appeler locationType ou juste type sur la table location
  • Les types lane / departmentalRoad devraient être renommés namedStreet / numberedRoad en cohérence avec les noms des tables

TODO

  • Comment héberger le serveur pg_featureserv en prod ? Créer une app séparée ?
  • Job ou trigger pour rafraîchir périodiquement la vue matérialisée
  • Doit-on accompagner l'API d'un schéma de données ?
  • Que faire des infos détaillées sur les périodes : jours concernés (dailyRange), créneaux horaires (timeslots) ?
  • Ajouter une documentation
    • Faire une ADR ? Pour expliquer pourquoi pg_featureserv (alors qu'il y a d'autres options), avec avantages (léger, spécifique postgis, quasi aucune config) et inconvénients (pas bcp de contenu de debug sur le web / stackoverflow)

@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.80%. Comparing base (101be2f) to head (de850a7).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1062   +/-   ##
=========================================
  Coverage     98.80%   98.80%           
  Complexity     1739     1739           
=========================================
  Files           348      348           
  Lines          7551     7551           
=========================================
  Hits           7461     7461           
  Misses           90       90           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@florimondmanca florimondmanca added the not to merge Ne pas merger (preuve de concept, démo, cassé, autre raison...) label Nov 20, 2024
@mmarchois
Copy link
Collaborator

Cette PR est toujours d'actualité ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not to merge Ne pas merger (preuve de concept, démo, cassé, autre raison...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants