Skip to content

Fichiers - #4161

Merged
arnaudlevy merged 61 commits into
mainfrom
files-step-1
Jul 16, 2026
Merged

Fichiers#4161
arnaudlevy merged 61 commits into
mainfrom
files-step-1

Conversation

@arnaudlevy

@arnaudlevy arnaudlevy commented Jun 26, 2026

Copy link
Copy Markdown
Member

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

#4162

CRUD File et Category

Attention, plusieurs différences avec les médias :

  • les blobs doivent être privés (pas implémenté)
  • les blobs sont rattachés aux locas

https://developers.osuny.org/docs/admin/composants/media-library/

Dans cette étape, on s'arrête à l'infrastructure, pas encore de picker Vue.

Dans l'étape d'après, on généralisera les pickers (fichiers, images, personnes, organisations, actualités...).

À faire :

  • Mise à jour via file library

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

Co-authored-by: Sébastien Gaya <sebastien.gaya@gmail.com>
@qltysh

qltysh Bot commented Jun 26, 2026

Copy link
Copy Markdown

All good ✅

Comment thread app/models/communication/block/component/file.rb Outdated
Comment thread app/models/communication/file/localization/with_icon.rb Outdated
arnaudlevy and others added 6 commits July 4, 2026 11:18
* group by nature

* order

* move to first-level concerns

---------

Co-authored-by: Sébastien Gaya <sebastien.gaya@gmail.com>
@SebouChu

Copy link
Copy Markdown
Member

en fait, j'ai un problème avec le component file, dont la propriété id était lié à un ActiveStorage::Blob et maintenant un Communication::File. J'aimerai bien que ça soit un autre nom pour gérer la migration (genre communication_file_id)

@SebouChu

Copy link
Copy Markdown
Member

@arnaudlevy a mis à jour pour passer sur communication_file_id, j'ai adapté le service de migration en conséquence

@arnaudlevy
arnaudlevy merged commit 99d6f95 into main Jul 16, 2026
4 of 6 checks passed
@arnaudlevy
arnaudlevy deleted the files-step-1 branch July 16, 2026 15:47
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