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

[17.0][MIG] fs_image_thumbnail: Migration to 17.0 #346

Merged
merged 14 commits into from
May 31, 2024
Merged

[17.0][MIG] fs_image_thumbnail: Migration to 17.0 #346

merged 14 commits into from
May 31, 2024

Conversation

nguyenminhchien
Copy link

@nguyenminhchien nguyenminhchien commented Mar 1, 2024 •
edited

@nguyenminhchien nguyenminhchien force-pushed the 17.0-mig-fs_image_thumbnail branch 2 times, most recently from 9007413 to 6cac03c Compare March 1, 2024 03:48
@nguyenminhchien nguyenminhchien marked this pull request as ready for review March 1, 2024 03:51
@nguyenminhchien nguyenminhchien mentioned this pull request Mar 1, 2024
Open
14 tasks
@lmignon
Copy link
Sponsor Contributor

lmignon commented Mar 24, 2024

/ocabot migration fs_image_thumbnail

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 24, 2024
lmignon and others added 13 commits May 8, 2024 15:46
@lmignon @nguyenminhchien
[ADD] fs_image_thumbnail: A drop-in replacement of storage_thumbnail
ad3ea34
@nguyenminhchien
[UPD] Update fs_image_thumbnail.pot
ed698d3
@OCA-git-bot @nguyenminhchien
[BOT] post-merge updates
07bb814
@lmignon @nguyenminhchien
[FIX] fs_image_thumbnail: Keep images order in result of get_or_creat…
33c2a94 
…e_thumbnails
@OCA-git-bot @nguyenminhchien
[BOT] post-merge updates
a636dca
@Ivorra78 @nguyenminhchien
Added translation using Weblate (Spanish)
2124128
@Ivorra78 @nguyenminhchien
Translated using Weblate (Spanish)
85611eb 
Currently translated at 100.0% (27 of 27 strings)

Translation: storage-16.0/storage-16.0-fs_image_thumbnail
Translate-URL: https://translation.odoo-community.org/projects/storage-16-0/storage-16-0-fs_image_thumbnail/es/
@mymage @nguyenminhchien
Added translation using Weblate (Italian)
bbb61f9
@mymage @nguyenminhchien
Translated using Weblate (Italian)
ea262b6 
Currently translated at 100.0% (27 of 27 strings)

Translation: storage-16.0/storage-16.0-fs_image_thumbnail
Translate-URL: https://translation.odoo-community.org/projects/storage-16-0/storage-16-0-fs_image_thumbnail/it/
@mymage @nguyenminhchien
Translated using Weblate (Italian)
bac150d 
Currently translated at 100.0% (27 of 27 strings)

Translation: storage-16.0/storage-16.0-fs_image_thumbnail
Translate-URL: https://translation.odoo-community.org/projects/storage-16-0/storage-16-0-fs_image_thumbnail/it/
@sbejaoui @nguyenminhchien
[FIX] delete FsImageThumbnail on attachment delete
3c64914
@OCA-git-bot @nguyenminhchien
[BOT] post-merge updates
1710cc9
@nguyenminhchien
[IMP] fs_image_thumbnail: pre-commit auto fixes
dc9604d
@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-346-by-gurneyalex-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 21, 2024
@OCA-git-bot
Merge PR #346 into 17.0
ef9a267 
Signed-off-by gurneyalex
@OCA-git-bot
Copy link
Contributor

@gurneyalex your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-346-by-gurneyalex-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@gurneyalex
Copy link
Member

gurneyalex commented May 31, 2024 •
edited

merge failure should have been fixed by #373

I'm retrying

@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-346-by-gurneyalex-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 04116d5 into OCA:17.0 May 31, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 430bcf8. Thanks a lot for contributing to OCA. ❤️

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

@lmignon lmignon lmignon approved these changes

@gurneyalex gurneyalex gurneyalex approved these changes

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
17.0
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@nguyenminhchien @lmignon @gurneyalex @OCA-git-bot @Ivorra78 @mymage @sbejaoui