I've been testing the review and I think we need some documentation that describes a minimal requirements to setup the integration.
I just managed to be able to create an user from vuestorefront to tryton backend (and it's correctly created as web_user) but when I try to login I get the following error:
53808 140309286708992 [2020-02-27 14:32:40,288] INFO werkzeug 127.0.0.1 - - [27/Feb/2020 14:32:40] "POST /vuestorefront/web_shop_vue_storefront/vue_shop/user/login HTTP/1.1" 200 -
53808 140309286708992 [2020-02-27 14:32:40,299] INFO werkzeug 127.0.0.1 - - [27/Feb/2020 14:32:40] "OPTIONS /vuestorefront/web_shop_vue_storefront/vue_shop/user/me?token=9a31d85bee28bb1c1cea7a73b39f4d62e293192035526d04193690b00fa5a13b HTTP/1.1" 204 -
53808 140309286708992 [2020-02-27 14:32:40,315] ERROR trytond.protocols.wrappers _vue_storefront/vue_shop/user/me?token=9a31d85bee28bb1c1cea7a73b39f4d62e293192035526d04193690b00fa5a13b' [GET]>
Traceback (most recent call last):
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/protocols/wrappers.py", line 163, in wrapper
result = func(request, pool, *args, **kwargs)
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/modules/web_shop_vue_storefront/routes.py", line 25, in route
Session.reset(request.args['token'])
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/modules/web_user/user.py", line 489, in reset
cls.write(sessions, {})
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/model/modelsql.py", line 159, in wrapper
return func(cls, *args, **kwargs)
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/model/modelsql.py", line 984, in write
cursor.execute(*table.update(columns, update_values,
File "/home/pokoli/projectes/vuestorefront/trytond/trytond/backend/postgresql/database.py", line 67, in execute
cursor.execute(self, sql, args)
psycopg2.errors.ReadOnlySqlTransaction: cannot execute UPDATE in a read-only transaction
On 2020-06-04 13:27, Sergi Almacellas Abellana wrote:
> I'm wondering if we can separate this review into small parts:
>
> * sale_shiment_cost (this can be a separate issue)
> * web_shop (this will help to implement https://discuss.tryton.org/t/woocommerce-connector-for-tryton/2795/)
> * web_shop_vue_storefront
> * web_user
Not really because we need a reference implementation.
It was requested to rewrite the documentation part. I do not have time nor the resource to do it. So I will welcome any help if you want these modules in the next release.
I think it is unfair that for this modules we require more about the documentation than for all the modules added between 5.6 and 5.8. Especially since any existing modules have been updated to the new structure. I think we can enforce it only once a good percentage of existing modules have been migrated to the new doc structure.
So I'm going to publish these modules to be included in 5.8 as-is.