Skip to content

[Fichiers] Ranger les fichiers GEIQ dans un « dossier » S3 - #6486

Merged
celine-m-s merged 1 commit into
masterfrom
celinems/geiq-files
Jul 11, 2025
Merged

celine-m-s merged 1 commit into
masterfrom
celinems/geiq-files

Conversation

@celine-m-s

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Ils n'étaient pas dans un dossier alors que tous les autres fichiers de nos applications le sont.

🍰 Comment ?

Ajout du préfixe "geiq-assessment" à la clé.

@celine-m-s celine-m-s self-assigned this Jul 7, 2025
@celine-m-s celine-m-s added the tech label Jul 7, 2025
@celine-m-s
celine-m-s changed the base branch from master to celinems/files_s3_len July 7, 2025 14:08

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Est-ce qu’on migre aussi les fichiers existants vers le bon dossier ?

Comment thread itou/www/geiq_assessments_views/views.py Outdated
@celine-m-s
celine-m-s force-pushed the celinems/geiq-files branch from 825a300 to de5b824 Compare July 8, 2025 14:39
@celine-m-s
celine-m-s changed the base branch from celinems/files_s3_len to master July 8, 2025 14:40
@celine-m-s

celine-m-s commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor Author

Est-ce qu’on migre aussi les fichiers existants vers le bon dossier ?

Bonne remarque. Antoine s'est proposé de le faire ce matin. Sinon, ça ne me semble pas critique car les fichiers seront servis de toute manière.

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tout bon après un petit rebase.

@celine-m-s
celine-m-s force-pushed the celinems/geiq-files branch from de5b824 to 0fe916f Compare July 8, 2025 16:00
@celine-m-s
celine-m-s enabled auto-merge July 8, 2025 16:01
@celine-m-s
celine-m-s force-pushed the celinems/geiq-files branch from 0fe916f to acfd876 Compare July 11, 2025 12:36
@celine-m-s
celine-m-s force-pushed the celinems/geiq-files branch from acfd876 to 5e17724 Compare July 11, 2025 12:52
@celine-m-s
celine-m-s added this pull request to the merge queue Jul 11, 2025
Merged via the queue into master with commit 1f6955a Jul 11, 2025
@celine-m-s
celine-m-s deleted the celinems/geiq-files branch July 11, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants