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.

Sign up for GitHub

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

Jump to bottom

[FEAT] Inform user about already registered telegram group #521

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[FEAT] Inform user about already registered telegram group #521

wants to merge 2 commits into from

Conversation

hv7214
Copy link

@hv7214 hv7214 commented Mar 28, 2020 •
edited
Loading

PR Summary

Now no group chatId is overwritten, and the new group is informed about the existence(if any) of the group name.

Screenshot from 2020-03-28 12-27-28

Additional information

Not sure if it is the right solution to the problem. But at least it solves the problem of overwriting of the existing group name chatId.

Closes/Relates issue

#373

@hv7214 hv7214 requested a review from a team as a code owner March 28, 2020 07:09
@coveralls
Copy link

coveralls commented Mar 28, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.007%) to 80.527% when pulling bc11dec on hv7214:fix/bad_redis_key_naming into 68d7c97 on moira-alert:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hv7214 @coveralls