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.0][MIG]website_event_require_login: Migration to 15.0 #290

Conversation

bizzappdev
Copy link

No description provided.

chienandalu and others added 20 commits January 2, 2023 11:22
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: event-12.0/event-12.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-12-0/event-12-0-website_event_require_login/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: event-12.0/event-12.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-12-0/event-12-0-website_event_require_login/
[UPD] Update website_event_require_login.pot

[UPD] README.rst
Wrong call of render_template() .  _render_template() converted a private function in standard Odoo v14.
Currently translated at 90.9% (10 of 11 strings)

Translation: event-14.0/event-14.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-website_event_require_login/pt/
Currently translated at 36.3% (4 of 11 strings)

Translation: event-14.0/event-14.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-website_event_require_login/it/
Currently translated at 45.4% (5 of 11 strings)

Translation: event-14.0/event-14.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-website_event_require_login/it/
Currently translated at 63.6% (7 of 11 strings)

Translation: event-14.0/event-14.0-website_event_require_login
Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-website_event_require_login/it/
@bizzappdev bizzappdev changed the title [WIP][15.0][MIG]website_event_require_login: Migration to 15.0 [15.0][MIG]website_event_require_login: Migration to 15.0 Mar 4, 2023
@bizzappdev
Copy link
Author

Ready for Review

@pedrobaeza
Copy link
Member

/ocabot migration website_event_require_login

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Mar 4, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 4, 2023
16 tasks
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Where's the migration commit? 🤔

@bizzappdev bizzappdev changed the title [15.0][MIG]website_event_require_login: Migration to 15.0 [WIP][15.0][MIG]website_event_require_login: Migration to 15.0 Mar 6, 2023
@bizzappdev bizzappdev force-pushed the 15.0-mig-website_event_require_login-BAD-SIP branch from eb7b2ab to dae64b1 Compare March 6, 2023 11:36
@bizzappdev
Copy link
Author

@chienandalu sorry for confusion. Now we have separated the pre commit and migration commits

# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
{
"name": "Website Event Require Login",
"version": "15.0.1.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

As we are in a new version the module version should look like this

Suggested change
"version": "15.0.1.0.1",
"version": "15.0.1.0.0",

Copy link
Contributor

@stefan-tecnativa stefan-tecnativa left a comment

Choose a reason for hiding this comment

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

Functionality tested.👍️

Please squash administrative commits and all will be great.

@bizzappdev bizzappdev force-pushed the 15.0-mig-website_event_require_login-BAD-SIP branch from 8be21ef to 51d1d9f Compare May 11, 2023 10:35
@bizzappdev bizzappdev changed the title [WIP][15.0][MIG]website_event_require_login: Migration to 15.0 [15.0][MIG]website_event_require_login: Migration to 15.0 May 12, 2023
@bizzappdev
Copy link
Author

ready for the review

Copy link
Contributor

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

LG

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-290-by-rafaelbn-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2d305bf into OCA:15.0 May 16, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at af8bbce. 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