Skip to content

Conversation

@ChrisOForgeFlow
Copy link

@ChrisOForgeFlow ChrisOForgeFlow commented Apr 16, 2024

  • Refactor from storage_import_image_advanced
  • Change to use fs_storage and fs_product_multi_image as dependencies
  • Fix test to use properly test class dependency
  • Removed storage_id field, setup of model or fields is unique way to address fs in new model
  • Removed thumbnail support, because module is not required by module fs_product_multi_image, added a new glue module 'fs_import_image_advanced_thumbnail' support to create automatic thumbnails when module fs_image_thumbnail is installed

HviorForgeFlow and others added 28 commits April 15, 2024 12:15
@ChrisOForgeFlow ChrisOForgeFlow force-pushed the 16.0-mig-storage_import_image_advanced branch from 6dd56e7 to 0812682 Compare April 16, 2024 12:36
- Refactor from storage_import_image_advanced
- Change to user fs_storage and fs_product_multi_image as dependencies
- Fix test to use properly test class dependency
@ChrisOForgeFlow ChrisOForgeFlow force-pushed the 16.0-mig-storage_import_image_advanced branch from 0812682 to 0ccbfe6 Compare April 17, 2024 16:45
@ChrisOForgeFlow ChrisOForgeFlow changed the title [16.0][MIG-REF] storage_import_image_advanced -> fs_import_image_advanced [16.0][MIG-REF] storage_import_image_advanced -> fs_import_image_advanced, fs_import_image_advanced_thumbnail Apr 22, 2024
@ChrisOForgeFlow ChrisOForgeFlow marked this pull request as ready for review April 22, 2024 16:39
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 25, 2024
@github-actions github-actions bot closed this Sep 29, 2024
@AlexPForgeFlow
Copy link

can we reopen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants