-
-
Notifications
You must be signed in to change notification settings - Fork 695
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
[17.0][MIG] website_require_login: Migration to 17.0 #1038
Conversation
Thanks for the contribution. Please preserve commit history following technical method explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0. If the jump is between several versions, you have to modify the source branch in the main command to accommodate it to this circumstance. |
Okay, will try it out |
This reverts commit 8530bab.
This reverts commit da63a60.
Currently translated at 100.0% (13 of 13 strings) Translation: website-14.0/website-14.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-14-0/website-14-0-website_require_login/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/
Currently translated at 100.0% (13 of 13 strings) Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/
Currently translated at 61.5% (8 of 13 strings) Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/de/
@pedrobaeza This is okay? Or need something else? |
I'm afraid it's not correct, as the commit history is messed up starting with the addition of all files, then revert them, etc. And you should follow the rest of the guide for what to commit and with such names. It's better that you save the work, and start again from the scratch. Anyway, I'm just telling you about the formal things. I'm not a qualified reviewer for the module itself. |
Okay will close this PR and start over. |
Tested the Website Login Required addon on 17.0 and it works with no modification.