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

Corrections BDD et Demandes #597

Merged
merged 5 commits into from
Nov 18, 2021

Conversation

jusabatier
Copy link
Collaborator

Corrections :

  • Conserver les mêmes noms de table et séquences utilisés en BDD
  • Ajout de l'extension '.pdf' pour les fichiers générés lors d'une demande d'informations
  • Correction d'une erreur de génération de l'image dans les BP lors d'une demande d'information
  • Réduction des logs par défaut à INFO

Suite à : #593

@jusabatier
Copy link
Collaborator Author

Testé sur une Debian BullsEye avec TC9 et JAVA11.

@landryb
Copy link
Member

landryb commented Nov 17, 2021

sympa, merci @jusabatier ! c'était volontaire de passer hibernate.hbm2ddl.auto a none dans 8b4849c ?

pour 7aa93cf ca passe automatiquement le type de contenu a application/pdf pour que le navigateur reconnaisse que c'est du pdf et propose de l'ouvrir avec le handler correct ?

@pierrejego
Copy link
Member

@jusabatier Merci,

@landryb pour le 7aa93cf c'est juste qu'il manquait l'extension ça change rien au comportement. (enfin si ça marche maintenant :))

@jusabatier même question que Landry, pour le hibernate.hbm2ddl.auto, j'aime mieux le validate que le none. Tu l'as fait volontairement ?

@jusabatier
Copy link
Collaborator Author

c'était volontaire de passer hibernate.hbm2ddl.auto a none dans 8b4849c ?

Oui, ça causait des problèmes avec la nouvelle version de PostgeSQL (13).

De ce que je me rappelle avoir lu, les tables/sequences ne sont plus indexées de la même façon dans la BDD.
Du coup le validate plante l'application, mais avec none ça marche très bien.

Une autre solution aurait été de prendre une version plus récente du connecteur postgesql.

@pierrejego
Copy link
Member

Je merge, et je vois pour le changement de version de conncteur.

Merci !

@pierrejego pierrejego merged commit 8e1f599 into georchestra:issue-554-clean Nov 18, 2021
@MaelREBOUX
Copy link
Member

merci pour la contribution @jusabatier

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.

4 participants