Skip to content

[IMP] product_currency: migrate demo to Python via _load_records#899

Open
cav-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-t-64641-1-cav
Open

[IMP] product_currency: migrate demo to Python via _load_records#899
cav-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-t-64641-1-cav

Conversation

@cav-adhoc

@cav-adhoc cav-adhoc commented May 27, 2026

Copy link
Copy Markdown
Contributor
  • Replace XML demo record with a Python function called via
    in the demo manifest entry (native Odoo pattern)
  • Use BaseModel._load_records instead of a custom _create_demo_record:
    respects noupdate=True, survives -u without clobbering, no reaping
  • Add EUR demo product (product_with_forced_currency)
  • Add tests: USD/EUR force_currency assertions + idempotency check

@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@cav-adhoc cav-adhoc force-pushed the 19.0-t-64641-1-cav branch 6 times, most recently from 8e059ac to 3394614 Compare May 27, 2026 15:51
@cav-adhoc cav-adhoc force-pushed the 19.0-t-64641-1-cav branch 2 times, most recently from 47f9f3a to 931c3eb Compare June 12, 2026 18:54
@cav-adhoc cav-adhoc changed the title [IMP] product_currency: migrate demo to Python, add EUR demo, remove hook [IMP] product_currency: migrate demo to Python via _load_records Jun 12, 2026
@cav-adhoc cav-adhoc force-pushed the 19.0-t-64641-1-cav branch from 931c3eb to eb7b0b6 Compare June 12, 2026 18:58
- Replace XML demo record with a Python function called via <function>
  in the demo manifest entry (native Odoo pattern)
- Use BaseModel._load_records instead of a custom _create_demo_record:
  respects noupdate=True, survives -u without clobbering, no reaping
- Add EUR demo product (product_with_forced_currency)
- Add tests: USD/EUR force_currency assertions + idempotency check
@cav-adhoc cav-adhoc force-pushed the 19.0-t-64641-1-cav branch from eb7b0b6 to c689928 Compare June 12, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants