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.

Sign up for GitHub

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

Jump to bottom

Asset file migration for documents does not work for document #12732

Closed
mattiagiupponi opened this issue Nov 20, 2024 · 0 comments · Fixed by #12733 or #12746
Closed

Asset file migration for documents does not work for document #12732

mattiagiupponi opened this issue Nov 20, 2024 · 0 comments · Fixed by #12733 or #12746
Assignees
@mattiagiupponi
Labels
bug investigation
Milestone
5.0.0

Comments

@mattiagiupponi
Copy link
Contributor

mattiagiupponi commented Nov 20, 2024 •
edited
Loading

Expected Behavior

With the introduction of #12326 i should be able to move all my resources from the uploaded folder to the new asset folder

Actual Behavior

The document folder is entirely moved for the first resource. This will lead to the other resource to skip the location update to the new folder because the file does not exist anymore in the original position

Steps to Reproduce the Problem

  1. upgrade from 4.3.x to 4.4.x
  2. run the command Assets: implement migration for old uploaded files #12326

Specifications

  • GeoNode version: 4.4.x / master
  • Installation type (vanilla, geonode-project):
  • Installation method (manual, docker):
  • Platform:
  • Additional details:
@mattiagiupponi mattiagiupponi added this to the 5.0.0 milestone Nov 20, 2024
@mattiagiupponi mattiagiupponi self-assigned this Nov 20, 2024
mattiagiupponi added a commit that referenced this issue Nov 21, 2024
@mattiagiupponi
[Fixes #12732] Asset file migration for documents does not work for d…
77aeaef 
…ocument
mattiagiupponi added a commit that referenced this issue Nov 21, 2024
@mattiagiupponi
[Fixes #12732] Asset file migration for documents does not work for d…
cdc8ca6 
…ocument
@mattiagiupponi mattiagiupponi linked a pull request Nov 22, 2024 that will close this issue
[Fixes #12732] Asset file migration for documents does not work for d… #12733
Merged
13 tasks
github-actions bot pushed a commit that referenced this issue Nov 22, 2024
@mattiagiupponi @github-actions
[Fixes #12732] Asset file migration for documents does not work for d… (
ece5fda 
#12733)

* [Fixes #12732] Asset file migration for documents does not work for document

* [Fixes #12732] Asset file migration for documents does not work for document

(cherry picked from commit 0b792cf)
giohappy pushed a commit that referenced this issue Nov 25, 2024
@github-actions @mattiagiupponi
[Fixes #12732] Asset file migration for documents does not work for d… (
3a3c061 
#12733) (#12736)

* [Fixes #12732] Asset file migration for documents does not work for document

* [Fixes #12732] Asset file migration for documents does not work for document

(cherry picked from commit 0b792cf)

Co-authored-by: mattiagiupponi <[email protected]>
mattiagiupponi added a commit that referenced this issue Nov 26, 2024
@mattiagiupponi
Fixes #12732: destination must be a list
0b4fbc5 
even for a single file, the destination must be a list not a string
@mattiagiupponi mattiagiupponi linked a pull request Nov 26, 2024 that will close this issue
Fixes #12732: Files destination must be a list #12746
Merged
13 tasks
giohappy pushed a commit that referenced this issue Nov 26, 2024
@mattiagiupponi
Fixes #12732: destination must be a list (#12746)
993364a 
even for a single file, the destination must be a list not a string
github-actions bot pushed a commit that referenced this issue Nov 26, 2024
@mattiagiupponi @github-actions
Fixes #12732: destination must be a list (#12746)
6976196 
even for a single file, the destination must be a list not a string

(cherry picked from commit 993364a)
giohappy pushed a commit that referenced this issue Nov 26, 2024
@github-actions @mattiagiupponi
Fixes #12732: destination must be a list (#12746) (#12747)
27476dc 
even for a single file, the destination must be a list not a string

(cherry picked from commit 993364a)

Co-authored-by: mattiagiupponi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattiagiupponi mattiagiupponi

Labels
bug investigation
Projects
None yet
Milestone
5.0.0
1 participant
@mattiagiupponi