search

Cacti / plugin_reportit

ReportIt Plugin for Cacti
GNU General Public License v2.0
7 stars 9 forks source link

Unable to add email recipients to report #24

Closed lbsou closed 6 years ago

lbsou commented 6 years ago

Cacti 1.1.38 Reportit git clone develop branch 2018-06-28

When I try to add an email recipients in a report, I hit "add" then it just bring me to the list of report...

Nothing in the logs.

Another thing that is curious, at the bottom of the email tab, there is no "save" button like in other tabs.

Thank you!

rchen14b commented 6 years ago

Easy for you to say I am afraid to move to 1.2 yet way to many unknowns.. Hell I can't even compile spine after 1.18x my pollers stop polling..

@netniV

Haha, similar here. Since we can't affor to lose the stability,

ikorzha commented 6 years ago

SPINE 1.1.18 Copyright 2004-2017 by The Cacti Group

netniV commented 6 years ago

Oh. Well you should open an issue on the spine repo for that then. It shouldn't stop. But I'd wait for 1.2, it changes the pollers so that may help.

But using your DB and 1.1.38, I can confirm error 500... let me check out why.

netniV commented 6 years ago

OK and now we are back in action. Sorry about that.

netniV commented 6 years ago

Oh one other thing... for new issues, please create a new issue ;-) . Lets keep this thread to the bigger problem of trying to add recipients !

rchen14b commented 6 years ago

Oh one other thing... for new issues, please create a new issue ;-) . Lets keep this thread to the bigger problem of trying to add recipients !

Tried the new update, golden!

netniV commented 6 years ago

Does the view report bit all work OK too?

ikorzha commented 6 years ago

netniV, Error 500 has been fixed. However report viewing is broken. I will open a separate issue on it.

netniV commented 6 years ago

This has now been resolved and committed to the development code.

ikorzha commented 6 years ago

Tested this portion works thanks!

rchen14b commented 6 years ago

Thanks you.

Regards, Ruiyang(Ryan) Chen

On Oct 3, 2018, at 21:31, ikorzha notifications@github.com wrote:

Tested this portion works thanks!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

ikorzha commented 6 years ago

Just an FYI, I have opened issue #63 related to email delivery and archiving that is not currently functioning yet, even though email can now be successfully defined.