[19.0][OU-ADD] pos_self_order: Migration scripts - #5868
Conversation
OCA-git-bot
added
mod:openupgrade_scripts
2445f77 to
e777210
Compare
| # NOTHING TO DO | ||
|
|
||
| pos_self_order / pos.config / self_ordering_takeaway (boolean): DEL | ||
| # DONE: preserved in pre-migration.py and used in post-migration.py to conditionally link takeout preset. |
There was a problem hiding this comment.
I don't see why you need to copy this, you can just keep the original name
| """ | ||
| Link default presets to existing POS configurations | ||
| based on legacy self_ordering_takeaway setting. | ||
| """ |
There was a problem hiding this comment.
I think most of this should go to the migration script of pos_restaurant
|
/ocabot migration pos_self_order Depends on :
|
e777210 to
84c2cd4
Compare
84c2cd4 to
5276115
Compare
|
@hbrunn Thanks for the review! I have addressed both comments. I removed the unnecessary pre-migration.py from pos_self_order and kept the original self_ordering_takeaway column name for the post-migration. I also moved the preset linking logic to pos_restaurant, where pos.preset is defined. I have kept the pos_restaurant changes in a separate commit within this PR. I also tested the migration with different POS configurations and verified that the presets and fiscal positions are correctly assigned. Please let me know if you would prefer the pos_restaurant changes to be moved to a separate PR. |
4 participants
No description provided.