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

Clarification of notify comments and code #1037

Merged
merged 1 commit into from
Jun 12, 2021
Merged

Conversation

paskal
Copy link
Sponsor Collaborator

@paskal paskal commented Jun 7, 2021

Multiple small changes I wanted to cut off the telegram notification PR as they are not connected to it.

@paskal paskal requested a review from umputun June 7, 2021 20:03
@paskal paskal added the backend label Jun 7, 2021
@paskal paskal force-pushed the paskal/notify_improvements branch 3 times, most recently from f7ad276 to 6a344c6 Compare June 7, 2021 23:06
@paskal paskal force-pushed the paskal/notify_improvements branch from 6a344c6 to 4626ec7 Compare June 7, 2021 23:13
Copy link
Owner

@umputun umputun left a comment

Choose a reason for hiding this comment

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

LGTM

@umputun umputun merged commit ea15b28 into master Jun 12, 2021
@umputun umputun deleted the paskal/notify_improvements branch June 12, 2021 17:59
@paskal paskal added this to the v1.9 milestone Jan 15, 2022
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.

None yet

2 participants