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

feature: add matrix service #113

Merged
merged 7 commits into from
Apr 2, 2021
Merged

feature: add matrix service #113

merged 7 commits into from
Apr 2, 2021

Conversation

piksel
Copy link
Member

@piksel piksel commented Jan 11, 2021

Fixes #112

@codecov
Copy link

codecov bot commented Jan 11, 2021

Codecov Report

Merging #113 (de0d56c) into main (0f8a531) will increase coverage by 0.40%.
The diff coverage is 76.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   68.47%   68.87%   +0.40%     
==========================================
  Files          72       76       +4     
  Lines        2106     2262     +156     
==========================================
+ Hits         1442     1558     +116     
- Misses        547      572      +25     
- Partials      117      132      +15     
Impacted Files Coverage Δ
pkg/services/matrix/matrix.go 66.66% <66.66%> (ø)
pkg/services/matrix/matrix_client.go 70.17% <70.17%> (ø)
pkg/router/servicemap.go 100.00% <100.00%> (ø)
pkg/services/matrix/matrix_api.go 100.00% <100.00%> (ø)
pkg/services/matrix/matrix_config.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f8a531...de0d56c. Read the comment docs.

@piksel piksel added this to the v0.5.0 milestone Jan 23, 2021
@kubax
Copy link

kubax commented Feb 9, 2021

Hi,

is there any ETA on this? Some time ago it seems like watchtower added some check on the url and i can't push notifications to my matrix server because of that (previous it worked fine through a slack compatible webproxy bot, but it is not updates in month, and thus not compatible with the new[?] type of slack notifications)

i'd love to have this feature implemented...

@piksel piksel marked this pull request as ready for review March 6, 2021 22:47
@piksel piksel merged commit 28298f4 into main Apr 2, 2021
@piksel piksel deleted the feat/matrix branch April 2, 2021 14:18
piksel added a commit that referenced this pull request May 14, 2021
* feat(matrix): add matrix service
* test(matrix): add some matrix tests
* fix(matrix): simplify client using stateless APIs
* fix(matrix): slim the public API and fix lint errors
* test(matrix): add test case
* test(matrix): add erroring test case
piksel added a commit that referenced this pull request May 14, 2021
* feat(matrix): add matrix service
* test(matrix): add some matrix tests
* fix(matrix): simplify client using stateless APIs
* fix(matrix): slim the public API and fix lint errors
* test(matrix): add test case
* test(matrix): add erroring test case
@piksel piksel modified the milestones: v0.6.0, v0.5.0 Jul 1, 2021
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.

Feature request: Matrix notifications
2 participants