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

Refactoriser les photos de recensements #1264

Open
goulvench opened this issue May 24, 2024 · 0 comments
Open

Refactoriser les photos de recensements #1264

goulvench opened this issue May 24, 2024 · 0 comments

Comments

@goulvench
Copy link
Collaborator

Le fichier config/initializers/active_storage.rb contient des monkey-patches pour les objets ActiveStorage, qui font planter les tests à moins de lancer le script create_postgres_sequences_memoire_photos_numbers.rb.

Il serait possible de remplacer les champs ajoutés aux attachments par une table dédiée (recensement_photos ?), avec une foreign_key sur ActiveStorage::Attachment pour que les deux restent synchronisés.
Les photos seraient associées à ce modèle spécifique (RecensementPhoto ?) via has_one_attached :photo. Le lien avec le recensement serait has_many :recensement_photos et has_many :photos, through: :recensement_photos. Idéalement, cela permettrait de conserver la syntaxe recensement.photos.

Cela ne changerait rien à l'expérience utilisateur, mais cela éviterait les faux positifs à chaque lancement de tests, et on serait à l'abri des évolutions de l'API ActiveStorage.

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

No branches or pull requests

1 participant