Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

11.0 qr payments2 #1181

Open
wants to merge 330 commits into
base: 11.0
Choose a base branch
from

Conversation

KolushovAlexandr
Copy link
Contributor

No description provided.

Ivan Yelizariev and others added 30 commits July 12, 2018 16:00
…qr-payments-docs

Pos addons 11.0 pos qr payments docs
…os-qr-payments-merge-utf-and-cachier-issue
…qr-payments-merge-utf-and-cachier-issue

Pos addons 11.0 pos qr payments merge utf and cachier issue
…qr-payments-docs

Pos addons 11.0 pos qr payments docs
Ivan Yelizariev and others added 23 commits August 10, 2018 17:52
…os-qr-payments-new6

refactoring wechat and pos_wechat modules
…qr-payments-lints

Pos addons 11.0 pos qr payments lints
…dev#539)

* 📝 plan of new module

* 🚒 add test_send_message

* 🚒 test_send_message

* 🚒 fix test_send_message

* 📝 fix doc

* 🚒 fix test_send_message

* 🚒 fix test_send_message

* 🚒 fix patch for test_send_message

* 🚒 new test_send_group_message

* 🔒 add rules

* 💚

* 🔥 fix depends

* 📝

* 💣 add new fields and refactoring code

* 💣 remove old code

* 🌹 change warning message

* 🔒 change access

* 📺 add view

* 📝 update usage instruction

* 📺 fix view

* 🔥 bool object has no attribute 'split'

* 💣 return sms object

* 💣 return sms id

* 🚒 fix return values

* 🔥 refactoring tests

* 🚒 passing tests and refactoring

* 📺 fix view

* 🔥 fix tests

* 🚒 passing tests and refactoring code

* 🚒 passing tests

* 🌹 logs messages

* 📝 update usage instruction

* 📖 fix the format in index.rst
…-qr-payments-docs

📖 module presentation added
* 🍒 long name for manifest filed fixed

* 🍒 renaming (rule 25)
data['body'] = message['data']['order_short']

data['out_trade_no'] = (str(time.time()).replace('.', '') \
+ '{0:010}'.format(random.randint(1, 9999999999)) \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
the backslash is redundant between brackets

data['mch_id'] = request.env['ir.config_parameter'].get_param('wechat.mchId')
data['body'] = message['data']['order_short']

data['out_trade_no'] = (str(time.time()).replace('.', '') \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the backslash is redundant between brackets

import logging
import requests
import odoo
import json
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'json' imported but unused

Copy link
Contributor

@itpp-bot itpp-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

└─ requirements.txt

7 installable modules are updated:

├─ alipay/
|  ├─ README.rst
|  ├─ __init__.py
|  ├─ __manifest__.py
|  ├─ controllers/
|  |  ├─ __init__.py
|  |  └─ alipay_controllers.py
|  ├─ data/
|  |  ├─ ir_sequence_data.xml
|  |  └─ module_data.xml
|  ├─ doc/
|  |  ├─ changelog.rst
|  |  └─ index.rst
|  ├─ images/
|  |  └─ alipay.png
|  ├─ models/
|  |  ├─ __init__.py
|  |  ├─ account_journal.py
|  |  ├─ alipay_order.py
|  |  ├─ alipay_refund.py
|  |  ├─ ir_config_parameter.py
|  |  └─ res_users.py
|  ├─ security/
|  |  ├─ alipay_security.xml
|  |  └─ ir.model.access.csv
|  ├─ static/
|  |  └─ description/
|  |     ├─ alipay_tracking.png
|  |     ├─ icon.png
|  |     ├─ index.html
|  |     ├─ parameters.png
|  |     └─ sandbox.png
|  ├─ tests/
|  |  ├─ __init__.py
|  |  └─ test_alipay.py
|  └─ views/
|     ├─ account_journal_views.xml
|     ├─ account_menuitem.xml
|     ├─ alipay_order_views.xml
|     └─ alipay_refund_views.xml
├─ pos_alipay/
|  ├─ README.rst
|  ├─ __init__.py
|  ├─ __manifest__.py
|  ├─ doc/
|  |  ├─ changelog.rst
|  |  └─ index.rst
|  ├─ images/
|  |  └─ pos_alipay.png
|  ├─ models/
|  |  ├─ __init__.py
|  |  ├─ alipay_order.py
|  |  ├─ alipay_pos.py
|  |  └─ pos_config.py
|  ├─ security/
|  |  └─ alipay_security.xml
|  ├─ static/
|  |  ├─ description/
|  |  |  ├─ icon.png
|  |  |  ├─ index.html
|  |  |  ├─ pos_1.png
|  |  |  ├─ pos_2.png
|  |  |  └─ pos_3.png
|  |  └─ src/
|  |     ├─ js/
|  |     |  ├─ alipay.js
|  |     |  └─ tour.js
|  |     └─ xml/
|  |        └─ pos.xml
|  ├─ tests/
|  |  ├─ __init__.py
|  |  └─ test_alipay.py
|  ├─ views/
|  |  └─ assets.xml
|  └─ wizard/
|     ├─ __init__.py
|     ├─ pos_payment.py
|     └─ pos_payment_views.xml
├─ pos_mobile/
|  └─ static/
|     └─ src/
|        └─ js/
|           ├─ chrome.js
|           └─ screens.js
├─ pos_wechat_miniprogram/
|  ├─ README.rst
|  ├─ __init__.py
|  ├─ __manifest__.py
|  ├─ doc/
|  |  ├─ changelog.rst
|  |  └─ index.rst
|  ├─ images/
|  |  └─ pos_wechat_miniprogram.jpg
|  ├─ security/
|  |  ├─ ir.model.access.csv
|  |  └─ wechat_security.xml
|  └─ static/
|     └─ description/
|        └─ icon.png
├─ qcloud_sms/
|  ├─ Dockerfile
|  ├─ README.rst
|  ├─ __init__.py
|  ├─ __manifest__.py
|  ├─ doc/
|  |  ├─ changelog.rst
|  |  └─ index.rst
|  ├─ images/
|  |  └─ qcloud_sms1.jpg
|  ├─ models/
|  |  ├─ __init__.py
|  |  ├─ ir_config_parameter.py
|  |  └─ qcloud_sms.py
|  ├─ security/
|  |  └─ ir.model.access.csv
|  ├─ static/
|  |  └─ description/
|  |     └─ icon.png
|  ├─ tests/
|  |  ├─ __init__.py
|  |  └─ test_qcloud_sms.py
|  └─ views/
|     └─ res_config.xml
├─ qr_payments/
|  └─ tools/
|     └─ async.py
└─ wechat_miniprogram/
   ├─ README.rst
   ├─ __init__.py
   ├─ __manifest__.py
   ├─ controllers/
   |  ├─ __init__.py
   |  └─ wechat_controllers.py
   ├─ data/
   |  └─ module_data.xml
   ├─ doc/
   |  ├─ changelog.rst
   |  └─ index.rst
   ├─ images/
   |  └─ wechat_miniprogram.jpg
   ├─ models/
   |  ├─ __init__.py
   |  ├─ ir_config_parameter.py
   |  ├─ res_users.py
   |  └─ wechat_order.py
   ├─ security/
   |  ├─ ir.model.access.csv
   |  └─ wechat_security.xml
   ├─ static/
   |  └─ description/
   |     └─ icon.png
   └─ tests/
      ├─ __init__.py
      └─ test_wechat_order.py

2 not installable modules s are updated:

├─ payment_wechat/
|  ├─ README.rst
|  ├─ __init__.py
|  ├─ __manifest__.py
|  ├─ controllers/
|  |  ├─ __init__.py
|  |  └─ p_w_controllers.py
|  ├─ demo/
|  |  └─ w_p_demo.xml
|  ├─ doc/
|  |  ├─ changelog.rst
|  |  └─ index.rst
|  ├─ models/
|  |  ├─ __init__.py
|  |  └─ wechat_models.py
|  ├─ static/
|  |  └─ description/
|  |     └─ icon.png
|  └─ views/
|     └─ views.xml
└─ pos_payment/
   ├─ README.rst
   ├─ __init__.py
   ├─ __manifest__.py
   ├─ doc/
   |  ├─ changelog.rst
   |  └─ index.rst
   └─ static/
      └─ description/
         └─ icon.png

sent by ✌️ Odoo Review Bot

`1.0.0`
-------

- Init version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

`1.0.0`
-------

- Init version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

`1.0.0`
-------

- Init version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

`1.0.0`
-------

- Init version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

`1.0.0`
-------

- Init version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

try:
# python 3
return method.__self__
except:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do not use bare 'except'

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.

3 participants