[16.0][OU-IMP] point_of_sale: Complete migration scripts #4417
Conversation
|
FYI @carmenbianca |
| env.cr.execute("SELECT id FROM survey_survey") | ||
| for row in env.cr.fetchall(): | ||
| env.cr.execute( | ||
| "UPDATE survey_survey SET access_token = %s WHERE id = %s", | ||
| (str(uuid.uuid4()), row[0]), | ||
| ) |
There was a problem hiding this comment.
I installed point of sale in a V15 database and a V16 database, and in each case, the request fail SELECT id FROM survey_survey;
ERREUR: la relation « survey_survey » n'existe pas
LIGNE 1 : SELECT id FROM survey_survey;
That is quite normal as survey.survey is defined in survey module that is not a dependency of point_of_sale module.
- Could you elaborate why you introduced that code ?
2) I can not understand why the CI is green.
EDIT : The CI is green, because survey is marked as done, so it is installed by the CI.
There was a problem hiding this comment.
Ouch, copy/paste...
Fixing it. The problem is that the module was not put on the doc (which I fixed in f52ca3f), and thus CI won't warn me, so now I'm reacting and merging all the needed pieces (as this didn't serve without pos_restaurant modules).
There was a problem hiding this comment.
Fixed in dea8b8a and merging pos_sale for normalizing CI.
|
Thanks @pedrobaeza ! you have an amazingly good eye for this |
@Tecnativa TT45176