Implement file copy for object storage when moving an issue
Closes #48505 (closed)
Merge request reports
Activity
added 1 commit
- 617516b0 - When moving issues, don't attempt to move files in object storage
@jprovaznik Could you take over this review? I'm not familiar with uploads(avatars/attachments/etc) so I don't have any idea about the right solution, here.
/cc @mbergeron
Hi,
Please add labels to your merge request, this helps triage community merge requests.
Thanks for your help!
added auto updated label
changed milestone to %11.1
changed milestone to %11.0
added ~4190206 regression labels
assigned to @jprovaznik
- Resolved by Micaël Bergeron
Thanks @stanhu, one comment inline.
assigned to @stanhu
assigned to @mbergeron
@jprovaznik can you review please, I've added the
FileUploader#copy_to
method that does the correct manipulation. When the file is stored remotely, I useCarrierWave::Fog::File#copy_to
method to clone the file directly so we don't have to download the file.I've been able to move issues with this fix, with uploads in OS or not (and mixed).
assigned to @jprovaznik
- Resolved by Micaël Bergeron