-
Notifications
You must be signed in to change notification settings - Fork 31
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
Weekly Report (yes, it's still not working) bug #26
Comments
No, that is the directory name in $HOME, and that is ok. There might be something wrong with your specific setup: |
that looks ok -I think- Could you run |
The last Weekly Report email I received was March 11th 2024, if that helps track down anything. Edit: To add. If you include this at the end of your URL the weekly reports are still being generated. It comes down to the apprise notification. localhost/views.php?view=Weekly%20Report&ascii=true |
@MueJosh help me understand the issue you're seeing is that
|
I don't even know how the weekly report should look like, is it a picture of the weekly_report.php or plain text? |
looking at the screenshot, I think you're seeing a division by zero issue. Could you update and run |
This could actually be progress :-) could you see what |
Ah yes, looking good. 👍 I'm not sure about the apprise error though... |
hmm, some googling sugests that this could be a Telegram limitation. |
I got fed up with the notifications / weekly report ;) Signal needs a docker / server to send messages, email needs the password (I have 2fa enabled- more issues) Problem are all the <>*/ ... on the weekly report pull request (never did this before; it took some time to figure it out) |
That is a great find! Seems the best solution I think |
Nice, thank you, @MueJosh. Interestingly, adding the OTH, it allows formatting for the body, which is great. Having scientific names in italics is such a relief... ;-) Here's my current take: ETA 2: 🦩?! 😂 |
@Nachtzuster there are definitely better ways- look at the pull request comment.
Yeah, this is exactly what markdown should do ^^ Yeah, this is really easy to implement if the format=markdown is just for the weeklyreport ...
What do you mean with this? Edit: I see... it works with a test notification but normal operation does not. I have an idea but no time to find out. I don't get the rest 😅 |
@Nachtzuster should I open another issue for notification breaks? Example, this is how it's set up normally: Code for break-line <> , /n , %0A or whatever the server wants for a break This is how it's supposed to look like (send with "test notification" button) Same break line If I have the time (can be weeks to months) I can handle it. Maybe it's an issue with apprise? (I don't think so) |
@MueJosh yes please. |
@MueJosh I've swapped out the |
In the settings menu for Apprise notification there is a problem with the weekly report.
It doesn't get send (this is also problem in the original repo)
To Reproduce
Steps to reproduce the behavior:
Tools
-Settings
-Notifications
send weekly report
Expected behavior
Get the weekly report as an notification from apprise
Screenshots
Your build
Raspberry Pi 4
64 bit bullseye lite
Code or log snippets
I think the problem lays in this script:
As Its says that the user is named Birdnet-pi and it should be the real username not the default one.
I can't test it right now but maybe you can get it working.
The text was updated successfully, but these errors were encountered: