-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Include assets inside B/R #12627
Comments
Two more tasks here, that can be implemented later:
|
mattiagiupponi
added a commit
that referenced
this issue
Oct 3, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
13 tasks
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 4, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 7, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 7, 2024
Restoring logger.info("*** Update Assets path...")
for instance in LocalAsset.objects.iterator():
should_be_updated = any(settings.ASSETS_ROOT not in loc for loc in instance.location)
if should_be_updated:
new_assets = []
for loc in instance.location:
if settings.ASSETS_ROOT not in loc:
new_assets.append(loc.replace(os.path.dirname(os.path.dirname(loc)), ttings.ASSETS_ROOT))
else:
new_assets.append(loc)
instance.location = new_assets
instance.save()
logger.info("*** Assets path updated...") |
mattiagiupponi
added a commit
that referenced
this issue
Oct 7, 2024
mattiagiupponi
added a commit
that referenced
this issue
Oct 7, 2024
* [Fixes #12627] Include assets inside B/R * [Fixes #12627] force localasset in get_link_url * [Fixes #12627] fix pr comments * [Fixes #12627] add todo for assets_root replace * [Fixes #12627] change utils assets_root name (cherry picked from commit 33be915) Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The assets (
ASSETS_FOLDER
) folder must be part of the B/R.The logic will be the same as for the media folder.
DRY, transform the logic in a common method, bot for the backup and restore.
The text was updated successfully, but these errors were encountered: