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.

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

[15][MIG] Migrate auto_backup #2253

Merged
merged 51 commits into from
Oct 1, 2022
Merged

Conversation

anhvu-sg
Copy link
Member

This module had been installed and used on my V15 project.

@anhvu-sg anhvu-sg mentioned this pull request Dec 31, 2021
54 tasks
@bobslee
Copy link

bobslee commented Jan 4, 2022

Hi @anhvu-sg , do yo know how this modules differs from? https://apps.odoo.com/apps/modules/15.0/auto_backup/
Seems there's already a v15 release for the one in the link.

Copy link
Contributor

@flachica flachica left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, squash commits

@anhvu-sg anhvu-sg force-pushed the 15.0-mig-auto-backup branch 2 times, most recently from 4a81b31 to 8bf701f Compare March 24, 2022 14:07
@anhvu-sg
Copy link
Member Author

Please, squash commits

done @flachica

@flachica
Copy link
Contributor

@sebalix @sbidoul This PR is Approved but no merged, Help, please

@cubells
Copy link
Sponsor Member

cubells commented Sep 14, 2022

@anhvu-sg you must add history of commits to this PR.

Yenthe666 and others added 19 commits September 14, 2022 13:36
Added all the new fields and sentences. This will be the template for
translations.
Used to open  fp = open(file_path,'wb') a few lines before it was needed. This shouldn't be too much of a problem but opening and closing it right after eachother keeps the session open for less time and there are less chances on failure.
- Follow template README.
- Remove HTML README.
- Move models to models folder.
- Model and view file names follow guidelines.
- Unused methods cleanup.
- Remove unneeded `.pot` file.
- Fix permissons.
- Follow PEP8 in names everywhere.
- Set more descriptive field names.
- Disable backups for other databases, for security.
- Remove db name from generated file, for easier cleanup.
- EAFP logic everywhere.
- More descriptive name.
- Data files moved to YAML, with cleaner ir.cron record creation.
- Add permissions for db.backup model.
- Icons.
- Update tests with new format.
- Storage method is a selectable, for easier extensibility.
- Instead of custom mailing, it just has a mail thread where you can subscribe.
- Should fix almost all comments in OCA#203.
- Reduce headers.

This respects the upstream license choice (GPL/AGPL) but reduces
verbosity.

It would be ideal to have everything under AGPL though.
* Add self.ensure_ones
* Add test coverage
* [10.0][auto_backup] Menu entry moved

* Update version in manifest
Currently translated at 58.2% (32 of 55 strings)

Translation: server-tools-11.0/server-tools-11.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-11-0/server-tools-11-0-auto_backup/nl/
oca-travis and others added 13 commits September 14, 2022 13:36
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/
Currently translated at 80.2% (69 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/it/
Currently translated at 86.0% (74 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/it/
Currently translated at 89.5% (77 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/it/
Currently translated at 94.1% (81 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/it/
Currently translated at 9.3% (8 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/ca/
Currently translated at 100.0% (86 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/es_AR/
Currently translated at 94.1% (81 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/it/
Currently translated at 50.0% (43 of 86 strings)

Translation: server-tools-14.0/server-tools-14.0-auto_backup
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-auto_backup/hr/
@anhvu-sg anhvu-sg force-pushed the 15.0-mig-auto-backup branch 2 times, most recently from 5d09af4 to 03a9ba2 Compare September 14, 2022 07:23
@anhvu-sg
Copy link
Member Author

hi @cubells , thanks for your support. The history have been updated

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@cubells
Copy link
Sponsor Member

cubells commented Sep 15, 2022

thanks @anhvu-sg

Now you can solve conflicts by rebasing

@oca-clabot
Copy link

Hey @anhvu-sg, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: https://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@anhvu-sg
Copy link
Member Author

thanks @anhvu-sg

Now you can solve conflicts by rebasing

done @cubells

@dreispt
Copy link
Sponsor Member

dreispt commented Oct 1, 2022

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 658c1c9 into OCA:15.0 Oct 1, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7befb85. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet