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

Add support for sender_chat (telegram) #1677

Merged
merged 2 commits into from
Feb 4, 2022
Merged

Add support for sender_chat (telegram) #1677

merged 2 commits into from
Feb 4, 2022

Conversation

42wim
Copy link
Owner

@42wim 42wim commented Jan 9, 2022

@42wim 42wim added this to the 1.24.0 milestone Jan 9, 2022
@@ -110,6 +110,23 @@ func (b *Btelegram) handleUsername(rmsg *config.Message, message *tgbotapi.Messa
}
}

if message.SenderChat != nil {
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@@ -110,6 +110,23 @@ func (b *Btelegram) handleUsername(rmsg *config.Message, message *tgbotapi.Messa
}
}

if message.SenderChat != nil { //nolint:nestif
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@codeclimate
Copy link

codeclimate bot commented Feb 3, 2022

Code Climate has analyzed commit cd8aeca and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@42wim 42wim merged commit 4b226a6 into master Feb 4, 2022
@42wim 42wim deleted the fix-1654 branch February 4, 2022 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Messages from Telegram channels appear as username "Telegram"
1 participant