Summary

Add Matrix option to Integration

Review Request #11195 — Created Sept. 21, 2020 and discarded Oct. 14, 2020, 12:03 p.m.

Information

Owner

ruonanjia

Repository

rbintegrations

Branch

master

Bugs

Depends On

Reviewers

Groups

rbintegrations, students

People

Description

Admins can choose to add Matrix Integration. This will send notifications to the registered matrix room about new review requets and updates to existing ones.

Testing Done

Tested in RB4.
Edited Slack unit test cases to test Matrix integration. All tests have passed except 2 of then complaining about missing image attachment
tests with errors are:
- test_notify_new_review_request_with_image_file_attachment
- test_notify_updated_review_request_with_new_image_attachments

Commits

Summary	ID	Author
add Matrix integration option	62db098bbb68cc97cee8d5e2890c6108e018ab6b	ruonan

Issues

Description	From	Last Updated
F401 'json' imported but unused	reviewbot	Oct. 14, 2020, 10:05 a.m.
F401 'django.utils.six.moves.urllib.request.Request' imported but unused	reviewbot	Oct. 14, 2020, 10:06 a.m.
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E303 too many blank lines (3)	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:07 a.m.
E303 too many blank lines (2)	reviewbot	Oct. 14, 2020, 10:07 a.m.
F401 'django.utils.six.moves.urllib.request.Request' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
F841 local variable 'response' is assigned to but never used	reviewbot	Oct. 14, 2020, 10:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:09 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:09 a.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 14, 2020, 10:10 a.m.
E501 line too long (82 > 79 characters)	reviewbot	Oct. 14, 2020, 10:11 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:11 a.m.
F401 'matrix_client.api.MatrixHttpApi' imported but unused	reviewbot	Oct. 14, 2020, 10:12 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:12 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:14 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:14 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:15 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:29 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 14, 2020, 10:16 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:17 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:17 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:18 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:21 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:26 a.m.
F821 undefined name 'self'	reviewbot	Oct. 14, 2020, 10:31 a.m.
E126 continuation line over-indented for hanging indent	reviewbot	Oct. 14, 2020, 10:31 a.m.
F821 undefined name 'json'	reviewbot	Oct. 14, 2020, 10:31 a.m.
F821 undefined name 'urlopen'	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (120 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (105 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (109 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:38 a.m.
E501 line too long (120 > 79 characters)	reviewbot	Oct. 14, 2020, 10:38 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:38 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:39 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:39 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (105 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (109 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (84 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (117 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (87 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (114 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 14, 2020, 10:50 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Oct. 14, 2020, 10:51 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Oct. 14, 2020, 10:51 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (88 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (114 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (90 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (98 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (91 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (84 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.

flake8 failed.

JSHint passed.

flake8

rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F401 'json' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F401 'django.utils.six.moves.urllib.request.Request' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```

Commits:

Summary	ID	Author
[WIP] - Add Matrix Chat option to Integration	352a64ace63a60611d8ba9e4f34c863993767206	ruonan
[WIP] - Add Matrix Chat option to Integration	352a64ace63a60611d8ba9e4f34c863993767206	ruonan
[WIP] implementing matrix communication	7567584c5a4107df57f51d9c76da4c3a5ee0aa6c	ruonan
[WIP] got matrix integration workinggit add .	ede798a8298879061ddc930cf0c4b1bf22fd03ba	ruonan

Diff:

Revision 2 (+3846 -16)

Show changes

	rbintegrations/extension.py
	rbintegrations/matrix/__init__.py
	rbintegrations/matrix/forms.py
	rbintegrations/matrix/integration.py
	rbintegrations/matrix/tests.py
	rbintegrations/static/images/matrix/icon.png

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

rbintegrations/matrix/forms.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
F401 'django.utils.six.moves.urllib.request.Request' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
F841 local variable 'response' is assigned to but never used
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```

Commits:

Summary	ID	Author
[WIP] - Add Matrix Chat option to Integration	352a64ace63a60611d8ba9e4f34c863993767206	ruonan
[WIP] implementing matrix communication	7567584c5a4107df57f51d9c76da4c3a5ee0aa6c	ruonan
[WIP] got matrix integration workinggit add .	ede798a8298879061ddc930cf0c4b1bf22fd03ba	ruonan
added Matrix integration option	02bcf3e41c18b01d6e3e11cfbfaeafd86b2f7d94	ruonan

Diff:

Revision 3 (+2596 -4)

Show changes

	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	rbintegrations/matrix/__init__.py
	rbintegrations/matrix/forms.py
	rbintegrations/matrix/integration.py
	rbintegrations/matrix/tests.py
	rbintegrations/static/images/matrix/icon.png

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 98 failures.
```
rbintegrations/basechat/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/forms.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (86 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (94 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (82 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
F401 'matrix_client.api.MatrixHttpApi' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (118 > 79 characters)
```

Summary:: [WIP] - Add Matrix Chat option to Integration
Add Matrix Chat option to Integration
Description:: ~
[WIP] - Add Matrix Chat option to Integration
~
Admins can choose to add Matrix Integration. This will send notifications to the registered matrix room about new review requets and updates to existing ones.
Testing Done:: +
Tested in RB4.
+ Edited Slack unit test cases to test Matrix integration. All tests have passed except 2 of then complaining about missing image attachment
+ tests with errors are:
+ - test_notify_new_review_request_with_image_file_attachment
+ - test_notify_updated_review_request_with_new_image_attachments

Commits:

	Summary	ID	Author
	added Matrix integration option	02bcf3e41c18b01d6e3e11cfbfaeafd86b2f7d94	ruonan
	add Matrix integration option	74667fd4436679bf110ca1d3a13770c02b70c41d	ruonan

Diff:

Revision 4 (+2692 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	docs/releasenotes/conf.py
	rbintegrations/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 90 failures.
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
F821 undefined name 'self'
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E126 continuation line over-indented for hanging indent
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
F821 undefined name 'json'
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
F821 undefined name 'urlopen'
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (83 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (112 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (113 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (106 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (120 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (112 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (113 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (106 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (99 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (105 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (109 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
The issue has been dropped. Show all issues
```
E501 line too long (108 > 79 characters)
```

Summary:: Add Matrix Chat option to Integration
Add Matrix option to Integration

Commits:

	Summary	ID	Author
	add Matrix integration option	74667fd4436679bf110ca1d3a13770c02b70c41d	ruonan
	add Matrix integration option	d7b7ad12cd24fa01bf0001c97d262e552e6f0562	ruonan

Diff:

Revision 5 (+2668 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 81 failures.
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E501 line too long (120 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (112 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (113 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (106 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (99 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (105 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (109 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (84 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (117 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (110 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (86 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (87 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (95 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
An issue was opened. Show all issues
```
E501 line too long (114 > 79 characters)
```

Commits:

	Summary	ID	Author
	add Matrix integration option	d7b7ad12cd24fa01bf0001c97d262e552e6f0562	ruonan
	add Matrix integration option	62db098bbb68cc97cee8d5e2890c6108e018ab6b	ruonan

Diff:

Revision 6 (+2712 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 144 failures.
```
rbintegrations/matrix/integration.py (Diff revision 6)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
rbintegrations/matrix/integration.py (Diff revision 6)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
rbintegrations/matrix/integration.py (Diff revision 6)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (94 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (95 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (88 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (114 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (86 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (99 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (90 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (98 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (91 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E501 line too long (84 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
An issue was opened. Show all issues
```
E128 continuation line under-indented for visual indent
```

Status:: Discarded

	+	Tested in RB4.
	+	Edited Slack unit test cases to test Matrix integration. All tests have passed except 2 of then complaining about missing image attachment
	+	tests with errors are:
	+	- test_notify_new_review_request_with_image_file_attachment
	+	- test_notify_updated_review_request_with_new_image_attachments