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

Remove no longer used channel_uuid and channel_type fields from msg events #529

Merged
merged 2 commits into from
Oct 17, 2022

Conversation

rowanseymour
Copy link
Member

In the end mailroom just uses channel_id

@codecov
Copy link

codecov bot commented Oct 17, 2022

Codecov Report

Merging #529 (e870be4) into main (ecb65dd) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #529      +/-   ##
==========================================
- Coverage   74.55%   74.54%   -0.01%     
==========================================
  Files          98       98              
  Lines       12448    12445       -3     
==========================================
- Hits         9280     9277       -3     
  Misses       2512     2512              
  Partials      656      656              
Impacted Files Coverage Δ
backends/rapidpro/task.go 67.39% <ø> (-1.03%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -1152,7 +1150,6 @@ func (ts *BackendTestSuite) TestWriteMsg() {
"text": msg.Text(),
"attachments": nil,
"new_contact": contact.IsNew_,
"created_on": msg.CreatedOn_.Format(time.RFC3339Nano),
Copy link
Member Author

Choose a reason for hiding this comment

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

@rowanseymour rowanseymour merged commit cb78ead into main Oct 17, 2022
@rowanseymour rowanseymour deleted the msg_queue_tweak branch October 17, 2022 20:46
@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant