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

Fix from RobertGnz about issue with too long messages (#275) #296

Closed
wants to merge 2 commits into from

Conversation

francwalter
Copy link

deleteMessage Funktion
README.md angepasst

deleteMessage Funktion
README.md angepasst
@witnessmenow
Copy link
Owner

At some point I will take a look at the solutions contained in this, but this isn't a PR that can be merged as is.

There is a lot of German in the comments and the readme is updated to reflect it is a fork

@francwalter
Copy link
Author

francwalter commented Apr 1, 2023

You live!
That's a start :)
Thanks!
Sorry for these German comments, I mixed something up with that PR I guess.
Will try a better, clean, PR with just the bugfix, but only in earliest 3 weeks (holidays).
Also the new delete function should be at separate PR and with English comments.

Edit 2024-03-06: I have never fixed my PR as promised last year!
But as this repo is not that vivid anyway and many PRs have never been pulled, I guess it is not really neccessary 😊

@francwalter francwalter closed this Mar 6, 2024
@francwalter
Copy link
Author

francwalter commented Mar 12, 2024

I just opened a new PR #348 with the fixed part and without my german comments. Hope this fits this time.

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.

2 participants