fix(tablet_pin_reset): manifest data slot + drop notif wrapper (deploy fixes)
Two bugs caught by entech battle test on first deploy: 1. Manifest entry landed in the 'demo' list instead of 'data' because my anchor (fp_demo_shopfloor_data.xml) was already in 'demo' — the entry pattern-matched into the wrong section. Demo data doesn't load on entech (no --load demo), so the mail.template never existed. Moved fp_tablet_pin_reset_template.xml to 'data'. 2. The fp.notification.template wrapper record referenced a model that doesn't exist until fusion_plating_notifications loads; fusion_plating_shopfloor doesn't depend on notifications, so the data load ParseError'd. Removed the wrapper — the controller calls mail_template.send_mail() directly anyway, not via the notification dispatcher. Added an inline comment explaining why the wrapper isn't here. Battle test updated to drop the (now removed) wrapper xmlid check. Battle test ALL PASS on entech after fixes. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -100,14 +100,7 @@ tpl = env.ref(
|
||||
)
|
||||
_ok(bool(tpl), 'mail.template fp_mail_template_tablet_pin_reset exists')
|
||||
|
||||
# 9. Notification template wrapper exists
|
||||
notif = env.ref(
|
||||
'fusion_plating_shopfloor.fp_notif_tablet_pin_reset',
|
||||
raise_if_not_found=False,
|
||||
)
|
||||
_ok(bool(notif), 'fp.notification.template fp_notif_tablet_pin_reset exists')
|
||||
|
||||
# 10. Cleanup cron exists
|
||||
# 9. Cleanup cron exists
|
||||
cron = env.ref(
|
||||
'fusion_plating_shopfloor.cron_purge_expired_pin_resets',
|
||||
raise_if_not_found=False,
|
||||
|
||||
Reference in New Issue
Block a user